diff options
author | eileencodes <eileencodes@gmail.com> | 2016-01-16 15:00:18 -0500 |
---|---|---|
committer | eileencodes <eileencodes@gmail.com> | 2016-01-16 15:00:18 -0500 |
commit | 5d1b7c3b441654e8008dcd303f5367883ec660a6 (patch) | |
tree | d3ded43da2dc15f5448a63d787f34413f26c7184 /guides/source/active_record_callbacks.md | |
parent | cbb7f7d00484a7bd90cf95f7c76741c19042b7df (diff) | |
download | rails-5d1b7c3b441654e8008dcd303f5367883ec660a6.tar.gz rails-5d1b7c3b441654e8008dcd303f5367883ec660a6.tar.bz2 rails-5d1b7c3b441654e8008dcd303f5367883ec660a6.zip |
Remove literal? check to fix issue with prefixed optionals
In commit d993cb3 `build_path` was changed from using `grep` to
`find_all` to save array allocations.
This change was a little too aggressive in that when the dash comes
before the symbol like `/omg-:song` the symbol is skipped.
Removing the check for `n.right.left.literal?` fixes this issue, but
does add back some allocations. The number of allocations are still well
less than before.
I've added a regression test to test this behavior for the future.
Fixes #23069.
Array allocations as of d993cb3:
```
{:T_SYMBOL=>11}
{:T_REGEXP=>17}
{:T_STRUCT=>6500}
{:T_MATCH=>12004}
{:T_OBJECT=>91009}
{:T_DATA=>100088}
{:T_HASH=>114013}
{:T_STRING=>159637}
{:T_ARRAY=>321056}
{:T_IMEMO=>351133}
```
Array allocations after this change:
```
{:T_SYMBOL=>11}
{:T_REGEXP=>1017}
{:T_STRUCT=>6500}
{:T_MATCH=>12004}
{:T_DATA=>84092}
{:T_OBJECT=>87009}
{:T_HASH=>110015}
{:T_STRING=>166152}
{:T_ARRAY=>322056}
{:T_NODE=>343558}
```
Diffstat (limited to 'guides/source/active_record_callbacks.md')
0 files changed, 0 insertions, 0 deletions