Add more tests, add more casts

Author: sweeneyde

Lib/test/test_range.py

@@ -507,7 +507,7 @@ def test_range_iterators(self):
         # exercise 'fast' iterators, that use a rangeiterobject internally.
         # see issue 7298
         limits = [base + jiggle
-                  for M in (2**32, 2**64)
+                  for M in (2**15, 2**30, 2**32, 2**64)
                   for base in (-M, -M//2, 0, M//2, M)
                   for jiggle in (-2, -1, 0, 1, 2)]
         test_ranges = [(start, end, step)
@@ -519,13 +519,14 @@ def test_range_iterators(self):
             iter1 = range(start, end, step)
             iter2 = pyrange(start, end, step)
             test_id = "range({}, {}, {})".format(start, end, step)
-            # check first 100 entries
-            self.assert_iterators_equal(iter1, iter2, test_id, limit=100)
+            # check first 10 entries
+            self.assert_iterators_equal(iter1, iter2, test_id, limit=10)
 
             iter1 = reversed(range(start, end, step))
             iter2 = pyrange_reversed(start, end, step)
             test_id = "reversed(range({}, {}, {}))".format(start, end, step)
-            self.assert_iterators_equal(iter1, iter2, test_id, limit=100)
+            self.assert_iterators_equal(iter1, iter2, test_id, limit=10)
+
 
     def test_range_iterators_invocation(self):
         # verify range iterators instances cannot be created by

Objects/rangeobject.c

@@ -1097,10 +1097,10 @@ range_iter(PyObject *seq)
         goto long_range;
     }
     ulen = get_len_of_range(lstart, lstop, lstep);
-    if (ulen > PyLong_MASK ||
-        lstart > PyLong_MASK || lstart < -(long)PyLong_MASK ||
-        lstop > PyLong_MASK || lstop < -(long)PyLong_MASK ||
-        lstep > PyLong_MASK || lstep < -(long)PyLong_MASK)
+    if (ulen > (long)PyLong_MASK ||
+        lstart > (long)PyLong_MASK || lstart < -(long)PyLong_MASK ||
+        lstop > (long)PyLong_MASK || lstop < -(long)PyLong_MASK ||
+        lstep > (long)PyLong_MASK || lstep < -(long)PyLong_MASK)
     {
         goto long_range;
     }
@@ -1190,10 +1190,10 @@ range_reverse(PyObject *seq, PyObject *Py_UNUSED(ignored))
 
     new_stop = lstart - lstep;
     new_start = (long)(new_stop + ulen * lstep);
-    if (ulen > PyLong_MASK ||
-        new_start > PyLong_MASK || new_start < -(long)PyLong_MASK ||
-        new_stop > PyLong_MASK || new_stop < -(long)PyLong_MASK ||
-        lstep > PyLong_MASK || lstep < -(long)PyLong_MASK)
+    if (ulen > (long)PyLong_MASK ||
+        new_start > (long)PyLong_MASK || new_start < -(long)PyLong_MASK ||
+        new_stop > (long)PyLong_MASK || new_stop < -(long)PyLong_MASK ||
+        lstep > (long)PyLong_MASK || lstep < -(long)PyLong_MASK)
     {
         goto long_range;
     }