From be5791be904582661051809d35dd86ef3831bbfe Mon Sep 17 00:00:00 2001 From: ydah <13041216+ydah@users.noreply.github.com> Date: Sat, 20 Jan 2024 00:54:31 +0900 Subject: [PATCH] Use reverse_each instead of reverse.each --- lib/lrama/context.rb | 2 +- lib/lrama/counterexamples/example.rb | 4 ++-- lib/lrama/grammar/code.rb | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/lrama/context.rb b/lib/lrama/context.rb index 245c91a1..5ca28179 100644 --- 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 index 8f02d71f..62244a77 100644 --- 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 index d0bef75e..e108d91c 100644 --- 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