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 289732 fda54bfb2896b2e513094ddc82ef5dde3fbc8f7d
parent 289731 cbbc3c971d809f28aa2075f8212836e0c3e84273
child 289733 6ffdefc8cba71642dd72887974a8a501e08d404b
push id30108
push usercbook@mozilla.com
push dateTue, 22 Mar 2016 11:14:31 +0000
treeherdermozilla-central@ea6298e1b4f7 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersterrence
bugs1258555
milestone48.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
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*> {};