Move Result to its own file (result.h) (#600)

* Unify all uses of `CHECK_RESULT`
* Use `CHECK_RESULT` in a few more places
* Add `operator|` and `operator|=` to `Result`, use it instead of
  `COMBINE_RESULT`
* Change a few occurrences of `!Succeeded` to `Failed`

1 files changed, 2 insertions, 11 deletions

diff --git a/src/binary-reader-interpreter.cc b/src/binary-reader-interpreter.cc
index 7cb3e676..ee51c07d 100644
--- a/src/binary-reader-interpreter.cc
+++ b/src/binary-reader-interpreter.cc
@@ -29,12 +29,6 @@
 #include "type-checker.h"
 #include "writer.h"
 
-#define CHECK_RESULT(expr)        \
-  do {                            \
-    if (Failed(expr))             \
-      return wabt::Result::Error; \
-  } while (0)
-
 namespace wabt {
 
 using namespace interpreter;
@@ -473,11 +467,8 @@ wabt::Result BinaryReaderInterpreter::GetBrDropKeepCount(
 wabt::Result BinaryReaderInterpreter::GetReturnDropKeepCount(
     Index* out_drop_count,
     Index* out_keep_count) {
-  if (Failed(GetBrDropKeepCount(label_stack.size() - 1, out_drop_count,
-                                out_keep_count))) {
-    return wabt::Result::Error;
-  }
-
+  CHECK_RESULT(GetBrDropKeepCount(label_stack.size() - 1, out_drop_count,
+                                  out_keep_count));
   *out_drop_count += current_func->param_and_local_types.size();
   return wabt::Result::Ok;
 }