remove extra loop, variable initialization #537

Merged
merged 1 commit into from Apr 12, 2013

Conversation

Projects
None yet
3 participants
Contributor

vipulnsward commented Apr 10, 2013

remove extra loop, variable initialization

Member

spastorino commented Apr 11, 2013

@raggi this is good to go, merge it

lib/rack/utils.rb
@@ -235,9 +235,7 @@ def select_best_encoding(available_encodings, accept_encoding)
end
expanded_accept_encoding.find_all { |m, q|
@rkh

rkh Apr 12, 2013

Member

Shouldn't the find_all now be an each?

Member

spastorino commented Apr 12, 2013

@vipulnsward actually there's no need to do a find_all just use each

Contributor

vipulnsward commented Apr 12, 2013

updating

Contributor

vipulnsward commented Apr 12, 2013

updated.

rkh added a commit that referenced this pull request Apr 12, 2013

Merge pull request #537 from vipulnsward/remove_loop
remove extra loop, variable initialization

@rkh rkh merged commit a210145 into rack:master Apr 12, 2013

1 check was pending

default The Travis build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment