Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Use Array#assoc in ActiveSupport::OrderedHash.

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@4817 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
  • Loading branch information...
commit 4dc464c1dfdba2241b54ccd60884e812db664d7c 1 parent ee23970
@jeremy jeremy authored
View
2  activesupport/CHANGELOG
@@ -1,5 +1,7 @@
*SVN*
+* Use Array#assoc in ActiveSupport::OrderedHash. [Mauricio Fernandez]
+
* Greatly increased performance of String.to_json, which speeds up RJS considerably on large pages, fixes #3473 [Shugo Maeda]
* Detect missing_constants calls from removed modules and fail accordingly. [Nicholas Seckar]
View
10 activesupport/lib/active_support/ordered_options.rb
@@ -2,7 +2,7 @@
module ActiveSupport
class OrderedHash < Array #:nodoc:
def []=(key, value)
- if pair = find_pair(key)
+ if pair = assoc(key)
pair.pop
pair << value
else
@@ -11,7 +11,7 @@ def []=(key, value)
end
def [](key)
- pair = find_pair(key)
+ pair = assoc(key)
pair ? pair.last : nil
end
@@ -22,12 +22,6 @@ def keys
def values
collect { |key, value| value }
end
-
- private
- def find_pair(key)
- self.each { |i| return i if i.first == key }
- return false
- end
end
end
Please sign in to comment.
Something went wrong with that request. Please try again.