diff options
author | Genadi Samokovarov <gsamokovarov@gmail.com> | 2013-07-01 12:14:09 +0300 |
---|---|---|
committer | Genadi Samokovarov <gsamokovarov@gmail.com> | 2013-07-01 12:18:25 +0300 |
commit | 269b4638d56081742a2166d23ead95d5c744aa48 (patch) | |
tree | dfe982f5dd5eecdd1facf9794a6458283c5436be /guides/source | |
parent | 18099b0672ff6df6c649f00d87a92ac28ce8d8e7 (diff) | |
download | rails-269b4638d56081742a2166d23ead95d5c744aa48.tar.gz rails-269b4638d56081742a2166d23ead95d5c744aa48.tar.bz2 rails-269b4638d56081742a2166d23ead95d5c744aa48.zip |
Further clean-up of ActiveSupport::Callbacks
In #11195 I noticed a trailing comma in the docs, but I decided to
further clean it up.
What I have done:
* Clean up the trailing comma in the docs and some extra whitespace
lines.
* Used `Array#extract` options to factor the repetitive pattern of
`args.last.is_a(Hash) ? ...`
* Renamed the local var `config` to `options` in `define_callbacks`, as
`options` seems to be the de facto name for the options objects.
* Renamed the local var `l` to `line` in `define_callback` (maybe it
meant `lambda` in the context) as single `l` may look like `1` in some
fonts.
Diffstat (limited to 'guides/source')
0 files changed, 0 insertions, 0 deletions