From de199bff9df9f42c164bf77ef501811eb2b0bbf4 Mon Sep 17 00:00:00 2001
From: "Anthony J. Thibault" <tony@highfidelity.io>
Date: Fri, 14 Jul 2017 09:47:37 -0700
Subject: [PATCH] code review feedback

---
 libraries/animation/src/AnimInverseKinematics.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/libraries/animation/src/AnimInverseKinematics.cpp b/libraries/animation/src/AnimInverseKinematics.cpp
index 20b62c2724..57c00e7183 100644
--- a/libraries/animation/src/AnimInverseKinematics.cpp
+++ b/libraries/animation/src/AnimInverseKinematics.cpp
@@ -930,7 +930,7 @@ const AnimPoseVec& AnimInverseKinematics::overlay(const AnimVariantMap& animVars
                     }
                 }
 
-                // identity joint chains that have changed types this frame.
+                // identify joint chains that have changed types this frame.
                 _prevJointChainInfoVec.resize(jointChainInfoVec.size());
                 for (size_t i = 0; i < _prevJointChainInfoVec.size(); i++) {
                     if (_prevJointChainInfoVec[i].timer <= 0.0f &&
@@ -944,7 +944,9 @@ const AnimPoseVec& AnimInverseKinematics::overlay(const AnimVariantMap& animVars
             {
                 PROFILE_RANGE_EX(simulation_animation, "ik/shiftHips", 0xffff00ff, 0);
 
-                if (_hipsTargetIndex >= 0 && _hipsTargetIndex >= 0 && _hipsTargetIndex < (int)targets.size()) {
+                if (_hipsTargetIndex >= 0) {
+                    assert(_hipsTargetIndex < (int)targets.size());
+
                     // slam the hips to match the _hipsTarget
 
                     AnimPose absPose = targets[_hipsTargetIndex].getPose();