Merge pull request #168 from yui-knk/initialize_ivars

Refactoring prec instance variables in parser
ruby · Oct 30, 2023 · f9d3772 · f9d3772
2 parents 116d364 + 9eff504
commit f9d3772
Show file tree

Hide file tree

Showing 2 changed files with 38 additions and 26 deletions.
diff --git a/lib/lrama/parser.rb b/lib/lrama/parser.rb
diff --git a/parser.y b/parser.y
@@ -292,15 +292,13 @@ rule
 
   rhs: /* empty */
          {
+           reset_precs
            result = []
-           @prec_seen = false
-           @code_after_prec = false
          }
      | "%empty"
          {
+           reset_precs
            result = []
-           @prec_seen = false
-           @code_after_prec = false
          }
      | rhs symbol named_ref_opt
          {
@@ -405,6 +403,7 @@ def parse
     @lexer = Lrama::Lexer.new(@text)
     @grammar = Lrama::Grammar.new
     @precedence_number = 0
+    reset_precs
     do_parse
     @grammar.extract_references
     @grammar.prepare
@@ -427,3 +426,10 @@ def on_error(error_token_id, error_value, value_stack)
     #{' ' * @lexer.column}^
   ERROR
 end
+
+private
+
+def reset_precs
+  @prec_seen = false
+  @code_after_prec = false
+end