Bug 1258555: Fix return type of js::GCPointerPolicy::needsSweep. r=terrence
authorJim Blandy <jimb@mozilla.com>
Mon, 21 Mar 2016 15:23:08 -0700
changeset 289791 fda54bfb2896b2e513094ddc82ef5dde3fbc8f7d
parent 289790 cbbc3c971d809f28aa2075f8212836e0c3e84273
child 289792 6ffdefc8cba71642dd72887974a8a501e08d404b
push id19656
push usergwagner@mozilla.com
push dateMon, 04 Apr 2016 13:43:23 +0000
treeherderb2g-inbound@e99061fde28a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersterrence
bugs1258555
milestone48.0a1
Bug 1258555: Fix return type of js::GCPointerPolicy::needsSweep. r=terrence
js/public/GCPolicyAPI.h
--- a/js/public/GCPolicyAPI.h
+++ b/js/public/GCPolicyAPI.h
@@ -86,17 +86,17 @@ template <> struct GCPolicy<uint64_t> : 
 template <typename T>
 struct GCPointerPolicy
 {
     static T initial() { return nullptr; }
     static void trace(JSTracer* trc, T* vp, const char* name) {
         if (*vp)
             js::UnsafeTraceManuallyBarrieredEdge(trc, vp, name);
     }
-    static void needsSweep(T* vp) {
+    static bool needsSweep(T* vp) {
         return js::gc::EdgeNeedsSweep(vp);
     }
 };
 template <> struct GCPolicy<JS::Symbol*> : public GCPointerPolicy<JS::Symbol*> {};
 template <> struct GCPolicy<JSAtom*> : public GCPointerPolicy<JSAtom*> {};
 template <> struct GCPolicy<JSFunction*> : public GCPointerPolicy<JSFunction*> {};
 template <> struct GCPolicy<JSObject*> : public GCPointerPolicy<JSObject*> {};
 template <> struct GCPolicy<JSScript*> : public GCPointerPolicy<JSScript*> {};