Merge pull request #546 from kostya/master

Array#reverse, reverse_each

lib-topaz/array.rb

@@ -298,4 +298,13 @@ def each_index(&block)
     0.upto(size - 1, &block)
     self
   end
+
+  def reverse
+    self.dup.reverse!
+  end
+
+  def reverse_each(&block)
+    reverse.each(&block)
+    self
+  end
 end

spec/tags/core/array/reverse_each_tags.txt

@@ -1,5 +1 @@
-fails:Array#reverse_each traverses array in reverse order and pass each element to block
-fails:Array#reverse_each returns self
-fails:Array#reverse_each yields only the top level element of an empty recursive arrays
-fails:Array#reverse_each yields only the top level element of a recursive array
 fails:Array#reverse_each returns an Enumerator if no block given

topaz/objects/arrayobject.py

@@ -279,3 +279,10 @@ def method_clear(self, space):
     def method_sort(self, space, block):
         RubySorter(space, self.items_w, sortblock=block).sort()
         return self
+
+    @classdef.method("reverse!")
+    @check_frozen()
+    def method_reverse_i(self, space):
+        self.items_w.reverse()
+        return self
+