Merge pull request RustPython#2077 from youknowone/opt-int

Optimize int.__new__

Author: youknowone

Lib/test/test_int.py

@@ -340,7 +340,8 @@ def test_int_memoryview(self):
     def test_string_float(self):
         self.assertRaises(ValueError, int, '1.2')
 
-    @unittest.skip("TODO: RUSTPYTHON")
+    # TODO: RUSTPYTHON
+    @unittest.expectedFailure
     def test_intconversion(self):
         # Test __int__()
         class ClassicMissingMethods:

Lib/test/test_long.py

@@ -313,7 +313,6 @@ def check_format_1(self, x):
             with self.subTest(got=got):
                 self.assertEqual(int(got, 0), x)
 
-    @unittest.skip # TODO: RUSTPYTHON
     def test_format(self):
         for x in special:
             self.check_format_1(x)
@@ -378,7 +377,6 @@ def test_long(self):
         # See bpo-34087
         self.assertRaises(ValueError, int, '\u3053\u3093\u306b\u3061\u306f')
 
-    @unittest.expectedFailure # TODO: RUSTPYTHON
     def test_long_a(self):
         self.assertRaises(ValueError, int, '-012395', 0)
 
@@ -417,8 +415,9 @@ def check_float_conversion(self, n):
                "Got {}, expected {}.".format(n, actual, expected))
         self.assertEqual(actual, expected, msg)
 
+    # TODO: RUSTPYTHON
+    @unittest.expectedFailure
     #@support.requires_IEEE_754
-    @unittest.skip # TODO: RUSTPYTHON
     def test_float_conversion(self):
 
         exact_values = [0, 1, 2,
@@ -830,8 +829,9 @@ def check_truediv(self, a, b, skip_small=True):
         self.assertEqual(expected, got, "Incorrectly rounded division {}/{}: "
                          "expected {}, got {}".format(a, b, expected, got))
 
+    # TODO: RUSTPYTHON
+    @unittest.expectedFailure
     #@support.requires_IEEE_754
-    @unittest.skip
     def test_correctly_rounded_true_division(self):
         # more stringent tests than those above, checking that the
         # result of true division of ints is always correctly rounded.

vm/src/bytesinner.rs

@@ -124,7 +124,7 @@ impl ByteInnerNewOptions {
 
                         let mut data_bytes = vec![];
                         for elem in elements {
-                            let v = objint::to_int(vm, &elem, &BigInt::from(10))?;
+                            let v = objint::to_int(vm, &elem)?;
                             if let Some(i) = v.to_u8() {
                                 data_bytes.push(i);
                             } else {