diff options
author | Rafael França <rafaelmfranca@gmail.com> | 2019-04-03 16:53:51 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-04-03 16:53:51 -0400 |
commit | 9d02b1bd58a20f34d36c461a0b88bfe590655b54 (patch) | |
tree | 45e6f472e112613b245b621933f5ae989c40d888 /actionpack/lib/action_dispatch/routing/mapper.rb | |
parent | 8113c8e1238939c8c68c2598b0a78a5b2171943d (diff) | |
parent | ed7234860b061b7b82672e6ba51c682254cd7eb7 (diff) | |
download | rails-9d02b1bd58a20f34d36c461a0b88bfe590655b54.tar.gz rails-9d02b1bd58a20f34d36c461a0b88bfe590655b54.tar.bz2 rails-9d02b1bd58a20f34d36c461a0b88bfe590655b54.zip |
Merge pull request #35842 from Shopify/deduplicate-routing-strings
Deduplicate strings held by the router
Diffstat (limited to 'actionpack/lib/action_dispatch/routing/mapper.rb')
-rw-r--r-- | actionpack/lib/action_dispatch/routing/mapper.rb | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/actionpack/lib/action_dispatch/routing/mapper.rb b/actionpack/lib/action_dispatch/routing/mapper.rb index 74596fa1f0..f29f66990d 100644 --- a/actionpack/lib/action_dispatch/routing/mapper.rb +++ b/actionpack/lib/action_dispatch/routing/mapper.rb @@ -115,9 +115,9 @@ module ActionDispatch @defaults = defaults @set = set - @to = to - @default_controller = controller - @default_action = default_action + @to = intern(to) + @default_controller = intern(controller) + @default_action = intern(default_action) @ast = ast @anchor = anchor @via = via @@ -222,6 +222,10 @@ module ActionDispatch private :build_path private + def intern(object) + object.is_a?(String) ? -object : object + end + def add_wildcard_options(options, formatted, path_ast) # Add a constraint for wildcard route to make it non-greedy and match the # optional format part of the route by default. |