simplified newrange.__reversed__ and corrected return type

Author: Brad Walker

src/future/types/newrange.py

@@ -107,12 +107,7 @@ def __contains__(self, value):
             return False
 
     def __reversed__(self):
-        """Return a range which represents a sequence whose
-        contents are the same as the sequence this range
-        represents, but in the opposite order."""
-        sign = self._step / abs(self._step)
-        last = self._start + ((self._len - 1) * self._step)
-        return newrange(last, self._start - sign, -1 * self._step)
+        return iter(self[::-1])
 
     def __getitem__(self, index):
         """Return the element at position ``index`` in the sequence

tests/test_future/test_range.py

@@ -6,12 +6,13 @@
 from future.builtins import range
 from future.tests.base import unittest
 
-from collections import Sequence
+from collections import Iterator, Sequence
 
 
 class RangeTests(unittest.TestCase):
     def test_range(self):
         self.assertTrue(isinstance(range(0), Sequence))
+        self.assertTrue(isinstance(reversed(range(0)), Iterator))
 
     def test_bool_range(self):
         self.assertFalse(range(0))