[GR-9749] List.index ends with IndexError instead ValueError.

Author: ppisl

graalpython/com.oracle.graal.python.test/src/tests/list_tests.py

@@ -361,7 +361,7 @@ def __eq__(self, other):
         for x, y in zip(d, e):
             # verify that original order and values are retained.
             self.assertIs(x, y)
-    '''
+
     def test_count(self):
         a = self.type2test([0, 1, 2])*3
         self.assertEqual(a.count(0), 3)
@@ -380,7 +380,7 @@ def __eq__(self, other):
                 return False
 
         self.assertRaises(BadExc, a.count, BadCmp())
-
+    
     def test_index(self):
         u = self.type2test([0, 1])
         self.assertEqual(u.index(0), 0)
@@ -437,7 +437,7 @@ def __eq__(self, other):
         a[:] = [EvilCmp(a) for _ in range(100)]
         # This used to seg fault before patch #1005778
         self.assertRaises(ValueError, a.index, None)
-
+        
     def test_reverse(self):
         u = self.type2test([-2, -1, 0, 1, 2])
         u2 = u[:]

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/list/ListBuiltins.java

@@ -808,8 +808,7 @@ private static int correctIndex(PList list, PInt index) {
         }
 
         private int findIndex(PList list, Object value, int start, int end, BinaryComparisonNode eqNode) {
-            int len = list.len();
-            for (int i = start; i < end && i < len; i++) {
+            for (int i = start; i < end && i < list.len(); i++) {
                 Object object = list.getItem(i);
                 if (eqNode.executeBool(object, value)) {
                     return i;