Changeset 4817

Timestamp:

08/25/06 06:01:02 (2 years ago)

Author:

bitsweat

Message:

Use Array#assoc in ActiveSupport::OrderedHash.

Files:

• trunk/activesupport/CHANGELOG (modified) (1 diff)
• trunk/activesupport/lib/active_support/ordered_options.rb (modified) (3 diffs)

trunk/activesupport/CHANGELOG

@@ -1 +1 @@
-*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]

trunk/activesupport/lib/active_support/ordered_options.rb

@@ -3 +3 @@
-  class OrderedHash < Array #:nodoc:
-    def []=(key, value)
-find_pair
+assoc
-        pair.pop
-        pair << value
@@ -12 +12 @@
-
-    def [](key)
-find_pair
+assoc
-      pair ? pair.last : nil
-    end
@@ -23 +23 @@
-      collect { |key, value| value }
-    end
-
-    private
-      def find_pair(key)
-        self.each { |i| return i if i.first == key }
-        return false
-      end
-  end
-end