From 5545da0648e27129ea41f96d26e2ee868eb5e667 Mon Sep 17 00:00:00 2001
From: Nick Fitzgerald <fitzgen@gmail.com>
Date: Fri, 17 Jun 2016 13:52:44 +0100
Subject: [PATCH] Bug 1267551 - Use MOZ_MUST_USE in
 js/public/UbiNodePostOrder.h; r=jimb

---
 js/public/UbiNodePostOrder.h | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/js/public/UbiNodePostOrder.h b/js/public/UbiNodePostOrder.h
index 7af382dbdf68..f0bf3878b1e9 100644
--- a/js/public/UbiNodePostOrder.h
+++ b/js/public/UbiNodePostOrder.h
@@ -7,6 +7,7 @@
 #ifndef js_UbiNodePostOrder_h
 #define js_UbiNodePostOrder_h
 
+#include "mozilla/Attributes.h"
 #include "mozilla/Maybe.h"
 #include "mozilla/Move.h"
 
@@ -90,7 +91,7 @@ struct PostOrder {
 #endif
 
   private:
-    bool fillEdgesFromRange(EdgeVector& edges, js::UniquePtr<EdgeRange>& range) {
+    MOZ_MUST_USE bool fillEdgesFromRange(EdgeVector& edges, js::UniquePtr<EdgeRange>& range) {
         MOZ_ASSERT(range);
         for ( ; !range->empty(); range->popFront()) {
             if (!edges.append(mozilla::Move(range->front())))
@@ -99,7 +100,7 @@ struct PostOrder {
         return true;
     }
 
-    bool pushForTraversing(const Node& node) {
+    MOZ_MUST_USE bool pushForTraversing(const Node& node) {
         EdgeVector edges;
         auto range = node.edges(rt, /* wantNames */ false);
         return range &&
@@ -124,11 +125,11 @@ struct PostOrder {
     { }
 
     // Initialize this traversal object. Return false on OOM.
-    bool init() { return seen.init(); }
+    MOZ_MUST_USE bool init() { return seen.init(); }
 
     // Add `node` as a starting point for the traversal. You may add
     // as many starting points as you like. Returns false on OOM.
-    bool addStart(const Node& node) {
+    MOZ_MUST_USE bool addStart(const Node& node) {
         if (!seen.put(node))
             return false;
         return pushForTraversing(node);
@@ -144,7 +145,7 @@ struct PostOrder {
     // Return false on OOM or error return from `onNode::operator()` or
     // `onEdge::operator()`.
     template<typename NodeVisitor, typename EdgeVisitor>
-    bool traverse(NodeVisitor onNode, EdgeVisitor onEdge) {
+    MOZ_MUST_USE bool traverse(NodeVisitor onNode, EdgeVisitor onEdge) {
 #ifdef DEBUG
         MOZ_ASSERT(!traversed, "Can only traverse() once!");
         traversed = true;