diff options
author | Luke Gruber <luke.gru@gmail.com> | 2014-06-25 23:18:33 -0400 |
---|---|---|
committer | Luke Gruber <luke.gru@gmail.com> | 2014-06-25 23:18:33 -0400 |
commit | 4955c04eb664e328597dae4ab5066e1b3021a614 (patch) | |
tree | 6c27cc767840ce03810d6b6434e31783cec57387 /activemodel/lib/active_model/callbacks.rb | |
parent | 18def156ba01f33255e09f18975c26b650975262 (diff) | |
download | rails-4955c04eb664e328597dae4ab5066e1b3021a614.tar.gz rails-4955c04eb664e328597dae4ab5066e1b3021a614.tar.bz2 rails-4955c04eb664e328597dae4ab5066e1b3021a614.zip |
remove Set.new from DetailsKey::get, impacts rendering overhead performance
Using ruby-prof, I noticed that Set#add had the largest 'self time'
percentage (5% of the overall time spent rendering) when
benchmarking the rendering of a small cached ERB template that was 3
lines long. It turns out it was from this line. I don't believe the
Set is necessary, either. Removing this line increases the rendering
ips using Benchmark::ips accordingly.
Diffstat (limited to 'activemodel/lib/active_model/callbacks.rb')
0 files changed, 0 insertions, 0 deletions