style fixes

- missing truffle boundary
- missing suppresswarnings annotation

Author: cosminbasca

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/PythonCextBuiltins.java

@@ -995,7 +995,7 @@ protected static boolean isPythonObjectNativeWrapper(Object object) {
     abstract static class CheckIterNextResultNode extends CheckFunctionResultNode {
 
         @Specialization(limit = "3")
-        static Object doGeneric(String name, Object result,
+        static Object doGeneric(@SuppressWarnings("unused") String name, Object result,
                         @CachedLibrary("result") InteropLibrary lib,
                         @CachedContext(PythonLanguage.class) ContextReference<PythonContext> contextRef,
                         @Cached PRaiseNode raiseNode) {

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext/CExtNodes.java

@@ -3281,7 +3281,7 @@ static long doOther(Object object,
         }
 
         static boolean isFallback(Object object) {
-            return !(object instanceof PInt || object instanceof Integer || object instanceof  Long);
+            return !(object instanceof PInt || object instanceof Integer || object instanceof Long);
         }
     }
 

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext/PyLongDigitsWrapper.java

@@ -142,7 +142,7 @@ final Object readArrayElement(long index,
                 }
                 return (val >> (longShift * index)) & longMask;
             } else {
-                byte[] bytes = ((PInt) delegate).abs().toByteArray();
+                byte[] bytes = PInt.toByteArray(((PInt) delegate).abs());
                 // the cast to int is safe since the length check already succeeded
                 return getUInt32(bytes, (int) index, longShift) & longMask;
             }

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext/common/CExtCommonNodes.java

@@ -541,7 +541,8 @@ static int doPIntTo32Bit(PInt obj, int signed, @SuppressWarnings("unused") long
             try {
                 if (signed != 0) {
                     return obj.intValueExact();
-                } else if (obj.bitLength() <= 32) {
+                } else if (obj.bitLength() <= 32) { // investigate the use of NarrowBigIntegerNode
+                                                    // (avoid the truffle boundary)
                     if (obj.isNegative()) {
                         return raiseNegativeValue(raiseNativeNode);
                     }

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/ints/PInt.java

@@ -487,7 +487,7 @@ public byte[] toByteArray() {
     }
 
     @TruffleBoundary
-    private static byte[] toByteArray(BigInteger delegate) {
+    public static byte[] toByteArray(BigInteger delegate) {
         return delegate.toByteArray();
     }
 

graalpython/lib-python/3/test/test_struct.py

@@ -198,6 +198,7 @@ def __init__(self, format):
             def test_one(self, x, pack=struct.pack,
                                   unpack=struct.unpack,
                                   unhexlify=binascii.unhexlify):
+
                 format = self.format
                 if self.min_value <= x <= self.max_value:
                     expected = x