Bug 1585519 - Part 4: Remove unnecessary parentheses. r=Yoric CLOSED TREE
authorTooru Fujisawa <arai_a@mac.com>
Mon, 07 Oct 2019 08:55:08 +0300
changeset 496486 011193f69f2fad43eeb92f597954b52d3e7755e8
parent 496485 b7dd0f12d98b0a00f61b7fab5c087932efd8f444
child 496487 05273fba6dadc286c4b61baa77cce7bf599e41c0
child 496617 3955e0a93047dec7ae5aebba4143aa61feed56ee
push id97271
push userbtara@mozilla.com
push dateMon, 07 Oct 2019 05:56:22 +0000
treeherderautoland@011193f69f2f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersYoric
bugs1585519
milestone71.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 1585519 - Part 4: Remove unnecessary parentheses. r=Yoric CLOSED TREE Reviewers: Yoric Subscribers: jandem Bug #: 1585519 Differential Revision: https://phabricator.services.mozilla.com/D48293
js/src/frontend/BinAST-macros.h
--- a/js/src/frontend/BinAST-macros.h
+++ b/js/src/frontend/BinAST-macros.h
@@ -20,28 +20,28 @@
   } while (false)
 
 // Evaluate an expression EXPR, checking that the result is not falsy.
 // In case of success, assign the result to VAR.
 //
 // Throw `cx->alreadyReportedError()` if it returns 0/nullptr.
 #define BINJS_TRY_VAR(VAR, EXPR)          \
   do {                                    \
-    (VAR) = (EXPR);                       \
+    VAR = (EXPR);                         \
     if (MOZ_UNLIKELY(!(VAR))) {           \
       return cx_->alreadyReportedError(); \
     }                                     \
   } while (false)
 
 // Evaluate an expression EXPR, checking that the result is not falsy.
 // In case of success, assign the result to a new variable VAR.
 //
 // Throw `cx->alreadyReportedError()` if it returns 0/nullptr.
 #define BINJS_TRY_DECL(VAR, EXPR)       \
-  auto (VAR) = (EXPR);                  \
+  auto VAR = (EXPR);                    \
   if (MOZ_UNLIKELY(!(VAR))) {           \
     return cx_->alreadyReportedError(); \
   }
 
 #define BINJS_TRY_EMPL(VAR, EXPR)           \
   do {                                      \
     auto _tryEmplResult = (EXPR);           \
     if (MOZ_UNLIKELY(!_tryEmplResult)) {    \
@@ -63,11 +63,11 @@
 // In case of success, unwrap and assign the result to a new variable VAR.
 //
 // In case of error, propagate the error.
 #define BINJS_MOZ_TRY_DECL(VAR, EXPR)          \
   auto _##VAR = (EXPR);                        \
   if (MOZ_UNLIKELY(_##VAR.isErr())) {          \
     return ::mozilla::Err(_##VAR.unwrapErr()); \
   }                                            \
-  auto (VAR) = _##VAR.unwrap();
+  auto VAR = _##VAR.unwrap();
 
 #endif  // frontend_BinAST_macros_h