Bug 603825 - Fix a bunch of warnings. r=jwalden.

js/src/jsinterp.cpp

@@ -448,7 +448,7 @@ ComputeGlobalThis(JSContext *cx, Value *argv)
 
 namespace js {
 
-bool
+void
 ReportIncompatibleMethod(JSContext *cx, Value *vp, Class *clasp)
 {
     Value &thisv = vp[1];
@@ -486,7 +486,6 @@ ReportIncompatibleMethod(JSContext *cx, Value *vp, Class *clasp)
                              clasp->name, JS_GetFunctionName(fun),
                              name);
     }
-    return false;
 }
 
 bool

js/src/jsinterp.h

@@ -805,7 +805,7 @@ namespace js {
  * Report an error that the this value passed as |this| in the given arguments
  * vector is not compatible with the specified class.
  */
-bool
+void
 ReportIncompatibleMethod(JSContext *cx, Value *vp, Class *clasp);
 
 /*

js/src/jsinterpinlines.h

@@ -633,7 +633,8 @@ GetPrimitiveThis(JSContext *cx, Value *vp, T *v)
         return true;
     }
 
-    return ReportIncompatibleMethod(cx, vp, Behavior::getClass());
+    ReportIncompatibleMethod(cx, vp, Behavior::getClass());
+    return false;
 }
 
 /*

js/src/methodjit/Compiler.cpp

@@ -2822,7 +2822,9 @@ mjit::Compiler::jsop_callprop_str(JSAtom *atom)
     frame.shift(-1);
 
     /* 4) Test if the function can take a primitive. */
+#ifdef DEBUG
     FrameEntry *funFe = frame.peek(-2);
+#endif
     JS_ASSERT(!funFe->isTypeKnown());
 
     /*