Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Perf optimization for `url_for` called w/ Hash | schneems | 2014-08-14 | 1 | -1/+3 |
| | | | | | | | | | | | | | | | | | | | | | Benchmarking the existing code: ```ruby { :only_path => options[:host].nil? }.merge!(options.symbolize_keys)) ``` Against optimized code, that does not require a new hash or a merge: ```ruby options = options.symbolize_keys options[:only_path] = options[:host].nil? unless options.key?(:only_path) options ``` We see a statistically significant performance gain: ![](https://www.dropbox.com/s/onocpc0zfw4kjxl/Screenshot%202014-08-14%2012.45.30.png?dl=1) Updated to not mutate incoming parameters | ||||
* | use the helper method builder to construct helper methods | Aaron Patterson | 2014-05-12 | 1 | -2/+6 |
| | |||||
* | probably should not check in a raise | Aaron Patterson | 2014-05-12 | 1 | -1/+0 |
| | |||||
* | undo optimized calls until the builder object is finished | Aaron Patterson | 2014-05-12 | 1 | -4/+3 |
| | |||||
* | fix url_for with classes | Aaron Patterson | 2014-05-12 | 1 | -1/+1 |
| | |||||
* | break each polymorphic type to it's own method | Aaron Patterson | 2014-05-12 | 1 | -1/+5 |
| | |||||
* | split nil / Hash cases in url_for | Aaron Patterson | 2014-04-30 | 1 | -4/+4 |
| | | | | | | | this reduces the number of comparisons and method calls `url_for` requires. The nil case no longer calls `symbolize_keys`, we already know options is nil, so no more ||=, and since it is nil we already know that options[:host] will be nil too. | ||||
* | Take Hash with options inside Array in #url_for | Andrey Ognevsky | 2013-11-15 | 1 | -0/+2 |
| | |||||
* | Move actionpack/lib/action_view* into actionview/lib | Piotr Sarnacki | 2013-06-20 | 1 | -0/+107 |