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.

Remove unnecessary line in the code

#4693

Looks like rack_test.rb has a line which is not needed and probably there by mistake. Attaching patch in the comment.

Reported by Anil Wadghule · May 25th, 2010 @ 07:49 PM

State: invalid
Milestone: none
Assigned to: nobody
Importance: none

Activity

  1. Anil Wadghule
  2. Anil Wadghule
    Anil Wadghule

    I came to know that request.request_parameters does params parsing. It forces normalize_params which does not happen on new as as you don't need to do (expensive) param parsing sometimes. It would error if params were parsed and the body was not rewound. (thanks @raggi for info)

    This ticket can be marked as invalid.

    May 25th, 2010 @ 08:33 PM

  3. Rohit Arondekar
    Rohit Arondekar
    • State changed from new to invalid

    June 15th, 2010 @ 01:47 PM