Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | [Active Support] require_relative => require | Akira Matsuda | 2017-10-21 | 1 | -1/+1 |
| | | | | This basically reverts 8da30ad6be34339124ba4cb4e36aea260dda12bc | ||||
* | [Active Support] `rubocop -a --only Layout/EmptyLineAfterMagicComment` | Koichi ITO | 2017-07-11 | 1 | -0/+1 |
| | |||||
* | Use frozen-string-literal in ActiveSupport | Kir Shatrov | 2017-07-09 | 1 | -0/+1 |
| | |||||
* | [Active Support] require => require_relative | Akira Matsuda | 2017-07-01 | 1 | -1/+1 |
| | |||||
* | applies new string literal convention in activesupport/lib | Xavier Noria | 2016-08-06 | 1 | -3/+3 |
| | | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default. | ||||
* | Fix broken number_to_currency tests | Prathamesh Sonpatki | 2016-01-29 | 1 | -0/+2 |
| | |||||
* | Change number_to_currency behavior for checking negativity | Prathamesh Sonpatki | 2016-01-29 | 1 | -5/+1 |
| | | | | | | | | | | | - Instead of using `to_f.phase`, just use `to_f.negative`?. - This change works same for all cases except when number is "-0.0". -0.0.to_f.negative? => false -0.0.to_f.phase? => pi - So -0.0 will be treated as positive from now onwards. - So this change reverts changes from https://github.com/rails/rails/pull/6512. - But it should be acceptable as we could not find any currency which supports negative zeros. | ||||
* | Give respond_to? a Symbol | Akira Matsuda | 2015-05-20 | 1 | -1/+1 |
| | |||||
* | Freeze static arguments for gsub | brainopia | 2015-04-02 | 1 | -1/+1 |
| | |||||
* | Prefer string patterns for gsub | brainopia | 2015-04-02 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | https://github.com/ruby/ruby/pull/579 - there is a new optimization since ruby 2.2 Previously regexp patterns were faster (since a string was converted to regexp underneath anyway). But now string patterns are faster and better reflect the purpose. Benchmark.ips do |bm| bm.report('regexp') { 'this is ::a random string'.gsub(/::/, '/') } bm.report('string') { 'this is ::a random string'.gsub('::', '/') } bm.compare! end # string: 753724.4 i/s # regexp: 501443.1 i/s - 1.50x slower | ||||
* | Doc fix [ci skip] | Sushruth Sivaramakrishnan | 2015-03-05 | 1 | -2/+2 |
| | |||||
* | Do gsub with a regexp instead of a string | Pablo Herrero | 2014-10-29 | 1 | -1/+1 |
| | |||||
* | Avoid a hash creation since defaults is a new hash already | Carlos Antonio da Silva | 2013-12-03 | 1 | -1/+1 |
| | |||||
* | Stop using local variables everywhere, make use of the reader | Carlos Antonio da Silva | 2013-12-03 | 1 | -2/+2 |
| | |||||
* | :sicssors: | Rafael Mendonça França | 2013-12-02 | 1 | -2/+0 |
| | |||||
* | Make execute priave API | Rafael Mendonça França | 2013-12-02 | 1 | -1/+1 |
| | |||||
* | Make load of NumberHelper thread safe | Rafael Mendonça França | 2013-12-02 | 1 | -0/+48 |