From 989b733de6abf64548afa9ec02f1d363e4402bac Mon Sep 17 00:00:00 2001
From: Andrew Meadows <andrew@highfidelity.io>
Date: Wed, 15 Jan 2014 16:59:49 -0800
Subject: [PATCH] Removing magic numbers, but still only supporting 2 hands
 (the SkeletonModel currently has two hands hardcoded).

---
 interface/src/avatar/Hand.cpp | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/interface/src/avatar/Hand.cpp b/interface/src/avatar/Hand.cpp
index 16693f88e2..b56e2c0c3d 100644
--- a/interface/src/avatar/Hand.cpp
+++ b/interface/src/avatar/Hand.cpp
@@ -617,8 +617,9 @@ void Hand::renderLeapHands(bool isMine) {
     }
 
     // Draw the hand paddles
+    int MAX_NUM_PADDLES = 2; // one for left and one for right
     glColor4f(handColor.r, handColor.g, handColor.b, 0.3f);
-    for (int i = 0; i < 2; i++) {
+    for (int i = 0; i < MAX_NUM_PADDLES; i++) {
         const PalmData* palm = getPalm(i);
         if (palm) {
             // compute finger axis
@@ -634,9 +635,9 @@ void Hand::renderLeapHands(bool isMine) {
             }
             // compute paddle position
             glm::vec3 handPosition;
-            if (i == 0) {
+            if (i == SIXENSE_CONTROLLER_ID_LEFT_HAND) {
                 _owningAvatar->getSkeletonModel().getLeftHandPosition(handPosition);
-            } else {
+            } else if (i == SIXENSE_CONTROLLER_ID_RIGHT_HAND) {
                 _owningAvatar->getSkeletonModel().getRightHandPosition(handPosition);
             }
             glm::vec3 tip = handPosition + HAND_PADDLE_OFFSET * fingerAxis;