Skip to content

Commit

Permalink
Merge pull request #522 from yui-knk/fix_comment
Browse files Browse the repository at this point in the history
Fix comments about `State#goto_follows`
  • Loading branch information
yui-knk authored Jan 22, 2025
2 parents 27f30e2 + 277a09e commit 88097af
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions lib/lrama/state.rb
Original file line number Diff line number Diff line change
Expand Up @@ -381,10 +381,7 @@ def goto_follow_set(nterm_token)
.reduce(always_follows(shift, next_state)) {|result, terms| result |= terms }
end

# Definition 3.14 (goto_follows, via successor_follows)
#
# TODO: This method uses `#always_follows` but the original papar uses `successor_follows`,
# is this correct?
# Definition 3.24 (goto_follows, via always_follows)
def goto_follows(shift, next_state)
queue = internal_dependencies(shift, next_state) + predecessor_dependencies(shift, next_state)
terms = always_follows(shift, next_state)
Expand Down

0 comments on commit 88097af

Please sign in to comment.