From 2a15726aafc2f4c27ee814cc571c54170d85ae0e Mon Sep 17 00:00:00 2001 From: Brad Davis Date: Thu, 10 Sep 2015 22:29:40 -0700 Subject: [PATCH] More PR comments --- libraries/physics/src/PhysicalEntitySimulation.cpp | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/libraries/physics/src/PhysicalEntitySimulation.cpp b/libraries/physics/src/PhysicalEntitySimulation.cpp index bee2a4d83d..19cfdddc6d 100644 --- a/libraries/physics/src/PhysicalEntitySimulation.cpp +++ b/libraries/physics/src/PhysicalEntitySimulation.cpp @@ -247,10 +247,12 @@ void PhysicalEntitySimulation::handleCollisionEvents(const CollisionEvents& coll void PhysicalEntitySimulation::addAction(EntityActionPointer action) { if (_physicsEngine) { // FIXME put fine grain locking into _physicsEngine - QMutexLocker lock(&_mutex); - const QUuid& actionID = action->getID(); - if (_physicsEngine->getActionByID(actionID)) { - qDebug() << "warning -- PhysicalEntitySimulation::addAction -- adding an action that was already in _physicsEngine"; + { + QMutexLocker lock(&_mutex); + const QUuid& actionID = action->getID(); + if (_physicsEngine->getActionByID(actionID)) { + qDebug() << "warning -- PhysicalEntitySimulation::addAction -- adding an action that was already in _physicsEngine"; + } } EntitySimulation::addAction(action); }