Fix style

Author: timfel

graalpython/com.oracle.graal.python.test/src/org/graalvm/python/embedding/cext/test/MultiContextCExtTest.java

@@ -214,7 +214,7 @@ public void testCreatingVenvForMulticontext() throws IOException {
         assertEquals("tiny_sha3", r2.asString());
         c2.eval("python", "import _sha3; _sha3.implementation = '12'");
         r2 = c2.eval(code);
-        assertEquals(".dup2", r2.asString());
+        assertEquals("12", r2.asString());
         // first context is unaffected
         r1 = c1.eval(code);
         assertEquals("tiny_sha3", r1.asString());

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

@@ -1049,7 +1049,7 @@ private static String dlopenFlagsToString(int flags) {
      * @param spec The name and path of the module (also containing the original module spec
      *            object).
      * @param checkFunctionResultNode A node to check that the function result does not indicate
-     *            that an exception was raised on the native side. It should be an adopted  node,
+     *            that an exception was raised on the native side. It should be an adopted node,
      *            because only an adopted node will report useful source locations.
      * @return A Python module.
      * @throws IOException If the specified file cannot be loaded.

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

@@ -80,7 +80,6 @@ public final class NativeLibraryLocator {
      */
     private static final AtomicInteger CEXT_COPY_INDICES = new AtomicInteger(MAX_CEXT_COPIES);
 
-
     /**
      * The suffix to add to C extensions when loading. This allows us to support native module
      * isolation with {@link PythonOptions#IsolateNativeModules}. If the value is {@code -1}, it
@@ -151,8 +150,8 @@ public void close() {
     public static void replicate(TruffleFile venvDirectory, PythonContext context, int count) throws IOException, InterruptedException {
         if (count > MAX_CEXT_COPIES) {
             LOGGER.warning(() -> String.format("The current limit for concurrent Python contexts accessing the Python C API is %d, " +
-                                            "but we are preparing %d copies. The extra copies will only be used if a different value " +
-                                            "of the system property %s is set.", MAX_CEXT_COPIES, count, J_MAX_CAPI_COPIES));
+                            "but we are preparing %d copies. The extra copies will only be used if a different value " +
+                            "of the system property %s is set.", MAX_CEXT_COPIES, count, J_MAX_CAPI_COPIES));
         }
         String suffix = context.getSoAbi().toJavaStringUncached();
         TruffleFile capiLibrary = context.getPublicTruffleFileRelaxed(context.getCAPIHome()).resolve(PythonContext.getSupportLibName("python-" + J_NATIVE));

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

@@ -42,8 +42,6 @@
 package com.oracle.graal.python.builtins.objects.cext.copying;
 
 import java.io.IOException;
-import java.io.OutputStream;
-import java.nio.file.CopyOption;
 import java.nio.file.StandardCopyOption;
 import java.nio.file.StandardOpenOption;
 

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

@@ -55,7 +55,7 @@ abstract class SharedObject implements AutoCloseable {
 
     abstract void write(TruffleFile copy) throws IOException, InterruptedException;
 
-    abstract public void close() throws IOException, InterruptedException;
+    public abstract void close() throws IOException, InterruptedException;
 
     static SharedObject open(TruffleFile file, PythonContext context) throws IOException {
         var f = file.readAllBytes();

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/ErrorMessages.java

@@ -1670,7 +1670,8 @@ public abstract class ErrorMessages {
     public static final TruffleString ARROW_ARRAY_ALREADY_RELEASED = tsLiteral("Cannot release already released ArrowArray");
     public static final TruffleString ARROW_SCHEMA_ALREADY_RELEASED = tsLiteral("Cannot release already released ArrowSchema");
     public static final TruffleString CAPI_ISOLATION_CAPPED_AT_D = tsLiteral(
-                    "There is no available slot for C API isolation. The current limit for concurrent Python contexts accessing the Python C API is %d. This can be changed with the" + J_MAX_CAPI_COPIES + " System property.");
+                    "There is no available slot for C API isolation. The current limit for concurrent Python contexts accessing the Python C API is %d. This can be changed with the" +
+                                    J_MAX_CAPI_COPIES + " System property.");
     public static final TruffleString SYS_PREFIX_MUST_BE_STRING_NOT_P_FOR_CAPI_ISOLATION = tsLiteral(
                     "The sys.prefix must be a str, not '%p' when the `IsolateNativeModules' option is used, because it is the base path for searching the relocated C API. " +
                                     "Refer to https://www.graalvm.org/latest/reference-manual/python/Native-Extensions for details on native module isolation.");

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/PythonOptions.java

@@ -348,8 +348,7 @@ private PythonOptions() {
     @Option(category = OptionCategory.EXPERT, usageSyntax = "true|false", help = "Whether C extension modules should be loaded as native code (as opposed to LLVM bitcode execution).") //
     public static final OptionKey<Boolean> NativeModules = new OptionKey<>(true);
 
-    @Option(category = OptionCategory.USER, stability = OptionStability.EXPERIMENTAL, usageSyntax = "true|false",
-                    help = "Whether the context should isolate its loading of C extension modules. " +
+    @Option(category = OptionCategory.USER, stability = OptionStability.EXPERIMENTAL, usageSyntax = "true|false", help = "Whether the context should isolate its loading of C extension modules. " +
                     "This allows more than one context to access C extensions. " +
                     "Note that all contexts in the operating system process must set this option to true to cooperatively allow this feature to work.") //
     public static final OptionKey<Boolean> IsolateNativeModules = new OptionKey<>(false);