From 8eacf0ad4b8579a5f6f2be561002a1020924a45c Mon Sep 17 00:00:00 2001 From: Ryan Huffman Date: Tue, 16 May 2017 10:56:33 -0700 Subject: [PATCH] Remove unnecessary assert in Model.cpp This clause inside was not longer valid - it is a valid state for part->_meshIndex < _modelMeshRenderItems.size(). A better check might be to compare it to the size of _meshStates which we use below, but we use _meshStates.at(...) which will do the bounds checking, which makes the assert unnecessary. --- libraries/render-utils/src/Model.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/libraries/render-utils/src/Model.cpp b/libraries/render-utils/src/Model.cpp index 766a584b85..3a4ec19d5c 100644 --- a/libraries/render-utils/src/Model.cpp +++ b/libraries/render-utils/src/Model.cpp @@ -1052,7 +1052,6 @@ void Model::updateRig(float deltaTime, glm::mat4 parentTransform) { void Model::computeMeshPartLocalBounds() { for (auto& part : _modelMeshRenderItems) { - assert(part->_meshIndex < _modelMeshRenderItems.size()); const Model::MeshState& state = _meshStates.at(part->_meshIndex); part->computeAdjustedLocalBound(state.clusterMatrices); }