Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

No need of extra check

#5734

no need to check for nil?

Besides the attachment I have some more fixes in 'nil_check' branch of forked rails.

http://github.com/neerajdotname/rails/commits/nil_check

Reported by Neeraj Singh · September 29th, 2010 @ 07:19 PM

State: committed
Milestone: 3.x
Assigned to: Aaron Patterson Aaron Patterson
Importance: Low

Activity

  1. Aaron Patterson
    Aaron Patterson
    • State changed from new to committed

    Thanks, applied!

    September 30th, 2010 @ 12:43 AM

  2. Neeraj Singh
    Neeraj Singh

    What do you think about other such fixes in this branch 'nil_check'

    http://github.com/neerajdotname/rails/commits/nil_check

    Thanks

    September 30th, 2010 @ 12:53 AM

  3. Aaron Patterson
    Aaron Patterson

    They seem good, so I've applied them.

    September 30th, 2010 @ 01:12 AM