Merge pull request #350 from ydah/reverse_each

Use reverse_each instead of reverse.each
ruby · Jan 22, 2024 · f6bf4eb · f6bf4eb
2 parents f884363 + be5791b
commit f6bf4eb
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/lib/lrama/context.rb b/lib/lrama/context.rb
@@ -221,7 +221,7 @@ def compute_yydefact
 
         if state.reduces.map(&:selected_look_ahead).any? {|la| !la.empty? }
           # Iterate reduces with reverse order so that first rule is used.
-          state.reduces.reverse.each do |reduce|
+          state.reduces.reverse_each do |reduce|
             reduce.look_ahead.each do |term|
               actions[term.number] = rule_id_to_action_number(reduce.rule.id)
             end

diff --git a/lib/lrama/counterexamples/example.rb b/lib/lrama/counterexamples/example.rb
@@ -40,7 +40,7 @@ def _derivations(paths)
         current = :production
         lookahead_sym = paths.last.to.item.end_of_rule? ? @conflict_symbol : nil
 
-        paths.reverse.each do |path|
+        paths.reverse_each do |path|
           item = path.to.item
 
           case current
@@ -97,7 +97,7 @@ def find_derivation_for_symbol(state_item, sym)
           if next_sym == sym
             derivation = nil
 
-            sis.reverse.each do |si|
+            sis.reverse_each do |si|
               derivation = Derivation.new(si.item, derivation)
             end
 

diff --git a/lib/lrama/grammar/code.rb b/lib/lrama/grammar/code.rb
@@ -28,7 +28,7 @@ def ==(other)
       def translated_code
         t_code = s_value.dup
 
-        references.reverse.each do |ref|
+        references.reverse_each do |ref|
           first_column = ref.first_column
           last_column = ref.last_column