kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

combine move_regexp and move_string so we only loop over states once

Browse source
This commit is contained in:
Aaron Patterson 2014-04-01 15:56:23 -07:00
parent ee453a1ea1
commit 0e53d11b1f
1 changed files with 8 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
actionpack/lib/action_dispatch/journey/gtg/transition_table.rb
View file

@ -41,7 +41,7 @@ module ActionDispatch
def move(t, a)
return [] if t.empty?
move_string(t, a).concat(move_regexp(t, a))
move_string(t, a)
end
def as_json(options = nil)
@ -141,20 +141,17 @@ module ActionDispatch
end
end
def move_regexp(t, a)
t.flat_map { |s|
if states = @regexp_states[s]
states.map { |re, v| re === a ? v : nil }
end
}.compact.uniq
end
def move_string(t, a)
t.map do |s|
regexps = []
t.map { |s|
if states = @regexp_states[s]
regexps.concat states.map { |re, v| re === a ? v : nil }
end
if states = @string_states[s]
states[a]
end
end.compact
}.compact.concat regexps
end
end
end