diff options
| author | Sean Griffin <sean@seantheprogrammer.com> | 2015-07-19 17:09:13 -0600 |
|---|---|---|
| committer | Sean Griffin <sean@seantheprogrammer.com> | 2015-07-19 17:09:13 -0600 |
| commit | f91439d848b305a9d8f83c10905e5012180ffa28 (patch) | |
| tree | 45c01f22da71b298dd2ea0ac3811adfb563d4ee5 /actionpack/lib/action_dispatch/journey/router/utils.rb | |
| parent | e19acbb8831e214eb38d589bf8c424edf80c2970 (diff) | |
| parent | 5bb1d4d288d019e276335465d0389fd2f5246bfd (diff) | |
| download | rails-f91439d848b305a9d8f83c10905e5012180ffa28.tar.gz rails-f91439d848b305a9d8f83c10905e5012180ffa28.tar.bz2 rails-f91439d848b305a9d8f83c10905e5012180ffa28.zip | |
Merge pull request #20946 from schneems/schneems/let-it-go
Freeze string literals when not mutated.
Diffstat (limited to 'actionpack/lib/action_dispatch/journey/router/utils.rb')
| -rw-r--r-- | actionpack/lib/action_dispatch/journey/router/utils.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/actionpack/lib/action_dispatch/journey/router/utils.rb b/actionpack/lib/action_dispatch/journey/router/utils.rb index d02ed96d0d..9793ca1c7a 100644 --- a/actionpack/lib/action_dispatch/journey/router/utils.rb +++ b/actionpack/lib/action_dispatch/journey/router/utils.rb @@ -14,10 +14,10 @@ module ActionDispatch # normalize_path("/%ab") # => "/%AB" def self.normalize_path(path) path = "/#{path}" - path.squeeze!('/') - path.sub!(%r{/+\Z}, '') + path.squeeze!('/'.freeze) + path.sub!(%r{/+\Z}, ''.freeze) path.gsub!(/(%[a-f0-9]{2})/) { $1.upcase } - path = '/' if path == '' + path = '/' if path == ''.freeze path end |