This project is archived and is in readonly mode.
Remove needless call to initializers
Reported by Ryan Bigg | February 7th, 2010 @ 11:53 PM | in 3.1
In the initializers method for Rails::Application,
the final line returns the collection of initializers,
as the line before it would do as well. This patch removes that
final line and will probably speed up the boot process of Rails by
0.000000000000001%.
Comments and changes to this ticket
-
Eadz February 8th, 2010 @ 12:25 AM
Thankyou for this patch. I have been waiting for this for so long! I hope this can make it into 3.0 :p
-
Rohit Arondekar October 8th, 2010 @ 02:16 AM
- State changed from “new” to “open”
- Assigned user set to “Ryan Bigg”
- Importance changed from “” to “Low”
I don't know how relevant this still is but that line is still there.
But do you think this needs tests? If not assign it to Santiago.
-
-
Santiago Pastorino October 8th, 2010 @ 11:40 AM
- Milestone set to “3.1”
- Assigned user changed from “Ryan Bigg” to “Santiago Pastorino”
Can you Ryan rebase this one, thx.
-
-
Repository October 8th, 2010 @ 06:34 PM
- State changed from “open” to “resolved”
(from [5826c1730720e8b11f7cc82831a1c76427e9ada2]) Remove the final initializers since initializers += will return it anyway. [#3892 state:resolved]
Signed-off-by: Santiago Pastorino santiago@wyeworks.com
http://github.com/rails/rails/commit/5826c1730720e8b11f7cc82831a1c7...
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
Eadz
Jeremy Kemper
Rohit Arondekar
Ryan Bigg
Santiago Pastorino
Attachments
Referenced by
-
3892
Remove needless call to initializers
(from [5826c1730720e8b11f7cc82831a1c76427e9ada2])
Remove ...