From 76a649b02b7dd88c439959011769fdac339fd2ef Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Vladim=C3=ADr=20Vondru=C5=A1?= Date: Sat, 11 Feb 2023 15:46:59 +0100 Subject: [PATCH] Trade: remove redundant assertions in SceneData::mutable*(). The two functions delegate to another mutable{Mapping,Field}(), which has the mutability assertion on its own, so it will fire nevertheless. --- src/Magnum/Trade/SceneData.cpp | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/Magnum/Trade/SceneData.cpp b/src/Magnum/Trade/SceneData.cpp index 7c8a2466b..28c8388cf 100644 --- a/src/Magnum/Trade/SceneData.cpp +++ b/src/Magnum/Trade/SceneData.cpp @@ -1294,8 +1294,6 @@ Containers::StridedArrayView2D SceneData::mapping(const SceneField f } Containers::StridedArrayView2D SceneData::mutableMapping(const SceneField fieldName) { - CORRADE_ASSERT(_dataFlags & DataFlag::Mutable, - "Trade::SceneData::mutableMapping(): data not mutable", {}); const UnsignedInt fieldId = findFieldIdInternal(fieldName); CORRADE_ASSERT(fieldId != ~UnsignedInt{}, "Trade::SceneData::mutableMapping(): field" << fieldName << "not found", {}); return mutableMapping(fieldId); @@ -1339,8 +1337,6 @@ Containers::StridedArrayView2D SceneData::field(const SceneField nam } Containers::StridedArrayView2D SceneData::mutableField(const SceneField name) { - CORRADE_ASSERT(_dataFlags & DataFlag::Mutable, - "Trade::SceneData::mutableField(): data not mutable", {}); const UnsignedInt fieldId = findFieldIdInternal(name); CORRADE_ASSERT(fieldId != ~UnsignedInt{}, "Trade::SceneData::mutableField(): field" << name << "not found", {});