This project is archived and is in readonly mode.

#5379 ✓committed
Prem Sichanugrist (sikachu)

[PATCH] Removing "symbol to proc" usage from Active Record

Reported by Prem Sichanugrist (sikachu) | August 14th, 2010 @ 12:06 AM | in 3.x

Hi,

I've seen the commits from Aaron Patterson on removing those symbol to proc (.map(&:method)) from the code to speeding up Active Record for a bit. I though it's a good idea, and wish I could help.

So, I've attached the patch which changes all the 'symbol to proc' in active_record/lib to a block. Noted that I've tried to use the variable name which is closest to the context. If it's inappropriate, please let me know

Thank you.

Comments and changes to this ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>

People watching this ticket

Attachments

Pages