Bug 1267551 (part 11) - Use MOZ_MUST_USE in js/src/vm/Printer.h. r=h4writer.

This catches a missing check.

--HG--
extra : rebase_source : 7a907da5a842ef0c4a677cbf17c86d8ee1d719db

js/src/jit/RangeAnalysis.cpp

@@ -1935,7 +1935,8 @@ RangeAnalysis::analyzeLoop(MBasicBlock* header)
 #ifdef DEBUG
     if (JitSpewEnabled(JitSpew_Range)) {
         Sprinter sp(GetJitContext()->cx);
-        sp.init();
+        if (!sp.init())
+            return false;
         iterationBound->boundSum.dump(sp);
         JitSpew(JitSpew_Range, "computed symbolic bound on backedges: %s",
                 sp.string());

js/src/vm/Printer.h

@@ -78,14 +78,14 @@ class Sprinter final : public GenericPrinter
     size_t                size;             // size of buffer allocated at base
     ptrdiff_t             offset;           // offset of next free char in buffer
 
-    bool realloc_(size_t newSize);
+    MOZ_MUST_USE bool realloc_(size_t newSize);
 
   public:
     explicit Sprinter(ExclusiveContext* cx, bool shouldReportOOM = true);
     ~Sprinter();
 
     // Initialize this sprinter, returns false on error.
-    bool init();
+    MOZ_MUST_USE bool init();
 
     void checkInvariants() const;
 
@@ -132,7 +132,7 @@ class Fprinter final : public GenericPrinter
     ~Fprinter();
 
     // Initialize this printer, returns false on error.
-    bool init(const char* path);
+    MOZ_MUST_USE bool init(const char* path);
     void init(FILE* fp);
     bool isInitialized() const {
         return file_ != nullptr;