diff options
| author | Xavier Noria <fxn@hashref.com> | 2011-07-08 00:58:14 +0200 |
|---|---|---|
| committer | Xavier Noria <fxn@hashref.com> | 2011-07-08 00:58:14 +0200 |
| commit | ad912c08a91b90f6a7b30ae75b4a3a5f6c513bd4 (patch) | |
| tree | 65d20f47dc4f2c08af7368100f859aa38eb34084 /actionpack/lib/action_view | |
| parent | e01aba15043fabf937bd6dbb818b6bce9a90651a (diff) | |
| download | rails-ad912c08a91b90f6a7b30ae75b4a3a5f6c513bd4.tar.gz rails-ad912c08a91b90f6a7b30ae75b4a3a5f6c513bd4.tar.bz2 rails-ad912c08a91b90f6a7b30ae75b4a3a5f6c513bd4.zip | |
use Zlib.crc2 rather that bytes.sum, as per Aaron's suggestion
That integer is rather irrelevant, the only thing that matters is that
it is consistent and with no apparent bias. Zlib.crc32 is 8-10 times
faster than bytes.sum, so use that.
Diffstat (limited to 'actionpack/lib/action_view')
| -rw-r--r-- | actionpack/lib/action_view/asset_paths.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/actionpack/lib/action_view/asset_paths.rb b/actionpack/lib/action_view/asset_paths.rb index 77a8c5322f..96d8fd0dfe 100644 --- a/actionpack/lib/action_view/asset_paths.rb +++ b/actionpack/lib/action_view/asset_paths.rb @@ -1,3 +1,4 @@ +require 'zlib' require 'active_support/core_ext/file' module ActionView @@ -111,8 +112,7 @@ module ActionView args << current_request if (arity > 1 || arity < 0) && has_request? host.call(*args) else - source_num = source.bytes.sum - (host =~ /%d/) ? host % (source_num % 4) : host + (host =~ /%d/) ? host % (Zlib.crc32(source) % 4) : host end end end |