rails/rails — comment on commit 563df87f19d21eb491f1c24dab0f59682fe0a737
File:  line 0
By ryanlecompte:

I've seen the options Hash#delete patten a lot, not sure why the option needs to be deleted instead of just using Hash#fetch with a default. :-/ Better to not muck with your caller's arguments if you can avoid it.

On Feb 20, 2012, at 9:02 PM, Joshua Clingenpeel wrote:

> Fetch leaves the k/v in the Hash, probably don't want that here. :(
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/rails/rails/commit/563df87f19d21eb491f1c24dab0f59682fe0a737#commitcomment-988165