Mirrors/overte
3
0
Fork 0
mirror of https://github.com/overte-org/overte.git synced 2025-08-08 10:17:40 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

remove assert in favor of warning

Browse source
This commit is contained in:
ZappoMan 2014-09-03 17:14:34 -07:00
parent 8223481ecb
commit d89b29581b
1 changed files with 9 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
libraries/entities/src/UpdateEntityOperator.cpp
View file

@ -175,7 +175,15 @@ bool UpdateEntityOperator::preRecursion(OctreeElement* element) {
// if we are the existing containing element, then we can just do the update of the entity properties // if we are the existing containing element, then we can just do the update of the entity properties
if (entityTreeElement == _containingElement) { if (entityTreeElement == _containingElement) {
assert(!_removeOld); // We shouldn't be in a remove old case and also be the new best fit
// TODO: We shouldn't be in a remove old case and also be the new best fit. This indicates that
// we have some kind of a logic error in this operator. But, it can handle it properly by setting
// the new properties for the entity and moving on. Still going to output a warning that if we
// see consistently we will want to address this.
if (_removeOld) {
qDebug() << "UNEXPECTED - UpdateEntityOperator - "
"we thought we needed to removeOld, but the old entity is our best fit.";
}
// set the entity properties and mark our element as changed. // set the entity properties and mark our element as changed.
_existingEntity->setProperties(_properties); _existingEntity->setProperties(_properties);