refactor s-expression parser

1 files changed, 8 insertions, 13 deletions

diff --git a/src/wasm-s-parser.h b/src/wasm-s-parser.h
index a181278ab..2456c5cca 100644
--- a/src/wasm-s-parser.h
+++ b/src/wasm-s-parser.h
@@ -105,12 +105,12 @@ class SExpressionParser {
 
 public:
   // Assumes control of and modifies the input.
-  SExpressionParser(char* input) : beginning(input), input(input) {}
-
-  Element* parseEverything() {
-    return parseInnerList();
+  SExpressionParser(char* input) : beginning(input), input(input) {
+    root = parseInnerList();
   }
 
+  Element* root;
+
 private:
   // parses the internal part of a list, inside the parens.
   Element* parseInnerList() {
@@ -181,18 +181,13 @@ class SExpressionWasmBuilder {
   Module& wasm;
 
   MixedArena allocator;
-  SExpressionParser parser;
 
 public:
   // Assumes control of and modifies the input.
-  SExpressionWasmBuilder(Module& wasm, char* input) : wasm(wasm), parser(input) {
-    Element* root = parser.parseEverything();
-    if (debug) std::cout << *root << '\n';
-    assert(root);
-    Element* module = (*root)[0];
-    assert((*module)[0]->str() == MODULE);
-    for (unsigned i = 1; i < module->size(); i++) {
-      parseModuleElement(*(*module)[i]);
+  SExpressionWasmBuilder(Module& wasm, Element& module) : wasm(wasm) {
+    assert(module[0]->str() == MODULE);
+    for (unsigned i = 1; i < module.size(); i++) {
+      parseModuleElement(*module[i]);
     }
   }