From 0b781a3589be3a3b604054f36e531714d7aff94a Mon Sep 17 00:00:00 2001
From: Brad Hefta-Gaub <brad@highfidelity.io>
Date: Thu, 17 Dec 2015 18:39:05 -0800
Subject: [PATCH 1/2] fix warnings

---
 interface/src/avatar/Avatar.cpp            | 2 +-
 interface/src/avatar/AvatarActionHold.cpp  | 2 +-
 interface/src/ui/ApplicationCompositor.cpp | 1 -
 3 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/interface/src/avatar/Avatar.cpp b/interface/src/avatar/Avatar.cpp
index f8040754d7..6094a0b9fe 100644
--- a/interface/src/avatar/Avatar.cpp
+++ b/interface/src/avatar/Avatar.cpp
@@ -157,7 +157,7 @@ void Avatar::animateScaleChanges(float deltaTime) {
 
         // snap to the end when we get close enough
         const float MIN_RELATIVE_SCALE_ERROR = 0.03f;
-        if (fabsf(_targetScale - currentScale) / _targetScale < 0.03f) {
+        if (fabsf(_targetScale - currentScale) / _targetScale < MIN_RELATIVE_SCALE_ERROR) {
             animatedScale = _targetScale;
         }
 
diff --git a/interface/src/avatar/AvatarActionHold.cpp b/interface/src/avatar/AvatarActionHold.cpp
index be562e2773..689d557c48 100644
--- a/interface/src/avatar/AvatarActionHold.cpp
+++ b/interface/src/avatar/AvatarActionHold.cpp
@@ -77,7 +77,7 @@ void AvatarActionHold::prepareForPhysicsSimulation() {
         // getTarget would get the palm position of the previous location of the avatar (because
         // bullet has moved the av's rigid body but the rigid body's location has not yet been
         // copied out into the Avatar class.
-        glm::quat avatarRotationInverse = glm::inverse(avatarRigidBodyRotation);
+        //glm::quat avatarRotationInverse = glm::inverse(avatarRigidBodyRotation);
 
         // the offset should be in the frame of the avatar, but something about the order
         // things are updated makes this wrong:
diff --git a/interface/src/ui/ApplicationCompositor.cpp b/interface/src/ui/ApplicationCompositor.cpp
index e5ecdfe217..abc3520c83 100644
--- a/interface/src/ui/ApplicationCompositor.cpp
+++ b/interface/src/ui/ApplicationCompositor.cpp
@@ -34,7 +34,6 @@
 static const quint64 MSECS_TO_USECS = 1000ULL;
 static const quint64 TOOLTIP_DELAY = 500 * MSECS_TO_USECS;
 
-static const float RETICLE_COLOR[] = { 0.0f, 198.0f / 255.0f, 244.0f / 255.0f };
 static const float reticleSize = TWO_PI / 100.0f;
 
 static const float CURSOR_PIXEL_SIZE = 32.0f;

From a7824a06d0b2d3bfa9d4fba2da1761d8e7bbf384 Mon Sep 17 00:00:00 2001
From: Brad Hefta-Gaub <brad@highfidelity.io>
Date: Thu, 17 Dec 2015 18:51:27 -0800
Subject: [PATCH 2/2] fix Assignment copy constructor warning

---
 libraries/networking/src/Assignment.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libraries/networking/src/Assignment.cpp b/libraries/networking/src/Assignment.cpp
index 4c72a16f1d..e6163c776b 100644
--- a/libraries/networking/src/Assignment.cpp
+++ b/libraries/networking/src/Assignment.cpp
@@ -91,7 +91,7 @@ Assignment::Assignment(ReceivedMessage& message) :
 #endif
 
 
-Assignment::Assignment(const Assignment& otherAssignment) {
+Assignment::Assignment(const Assignment& otherAssignment) : QObject() {
     _uuid = otherAssignment._uuid;
     _command = otherAssignment._command;
     _type = otherAssignment._type;