Fix style

Author: Pavel Marek

com.oracle.truffle.r.test/src/com/oracle/truffle/r/test/builtins/TestBuiltin_Sysgetenv.java

@@ -14,7 +14,7 @@
  * Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
  *
  * Copyright (c) 2012-2014, Purdue University
- * Copyright (c) 2013, 2020, Oracle and/or its affiliates
+ * Copyright (c) 2013, 2021, Oracle and/or its affiliates
  *
  * All rights reserved.
  */

com.oracle.truffle.r.test/src/com/oracle/truffle/r/test/generate/FastRContext.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2019, 2019, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2019, 2021, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it

com.oracle.truffle.r.test/src/com/oracle/truffle/r/test/generate/FastRSession.java

@@ -100,7 +100,8 @@ public static Source createSource(String txt, String name) {
         return Source.newBuilder("R", txt, name).internal(true).interactive(true).buildLiteral();
     }
 
-    // TODO: used from the outsize: can the users use the shared context? Otherwise change them to the exclusive one
+    // TODO: used from the outsize: can the users use the shared context? Otherwise change them to
+    // the exclusive one
     public FastRContext createContext(ContextKind contextKind) {
         return getContext(contextKind, true);
     }
@@ -162,7 +163,8 @@ private static FastRContext createVanillaContext(boolean isShared) {
         ByteArrayOutputStream output = new ByteArrayOutputStream();
         TestByteArrayInputStream input = new TestByteArrayInputStream();
         ChildContextInfo info = ChildContextInfo.create(params, null, ContextKind.SHARE_NOTHING, null, input, output, output);
-        // TODO: do we need the config info in this case? -- we should eventually remove it altogether
+        // TODO: do we need the config info in this case? -- we should eventually remove it
+        // altogether
         RContext.childInfo = info;
         Context truffleContext = getContextBuilder("R", "llvm").in(input).out(output).err(output).build();
         if (isShared) {
@@ -221,7 +223,7 @@ public String eval(TestBase testClass, String expression, ContextKind contextKin
         ByteArrayOutputStream output;
         try (FastRContext evalContext = getContext(contextKind, allowHostAccess)) {
             output = evalContext.getOutput();
-            eval(evalContext, testClass, expression, contextKind, timeout);
+            eval(evalContext, testClass, expression, timeout);
         }
         try {
             return output.toString("UTF-8");
@@ -231,7 +233,7 @@ public String eval(TestBase testClass, String expression, ContextKind contextKin
         }
     }
 
-    private void eval(FastRContext evalContext, TestBase testClass, String expression, ContextKind contextKind, long timeout) throws Throwable {
+    private void eval(FastRContext evalContext, TestBase testClass, String expression, long timeout) throws Throwable {
         try {
             evalContext.reset();
             evalContext.getInput().setContents(expression);

com.oracle.truffle.r.test/src/com/oracle/truffle/r/test/library/base/TestSimpleErrorHandling.java

@@ -32,9 +32,10 @@
 public class TestSimpleErrorHandling extends TestBase {
 
     /**
-     * Note that 'error' option cannot be simply reset in a standard R session.
-     * Running {@code old_opts <- options(); options(error = quote(cat('Nastala chyba\n'))); X; options(old_opts); X}
-     * does not produce the expected result, i.e., the second error message still contains 'Nastala chyba'.
+     * Note that 'error' option cannot be simply reset in a standard R session. Running
+     * {@code old_opts <- options(); options(error = quote(cat('Nastala chyba\n'))); X; options(old_opts); X}
+     * does not produce the expected result, i.e., the second error message still contains 'Nastala
+     * chyba'.
      */
     @Test
     public void testError() {

com.oracle.truffle.r.test/src/com/oracle/truffle/r/test/library/fastr/TestContextKind.java

@@ -24,8 +24,6 @@
 package com.oracle.truffle.r.test.library.fastr;
 
 import com.oracle.truffle.r.test.TestBase;
-import com.oracle.truffle.r.test.TestTrait;
-import org.junit.Ignore;
 import org.junit.Test;
 
 /**

com.oracle.truffle.r.test/src/com/oracle/truffle/r/test/library/stats/TestRandGenerationFunctions.java

@@ -26,7 +26,6 @@
 import java.util.HashSet;
 import java.util.Set;
 
-import org.junit.Ignore;
 import org.junit.Test;
 
 import com.oracle.truffle.r.test.TestBase;
@@ -125,8 +124,10 @@ public void testGenerators() {
 
     @Test
     public void testDotRandomSeed() {
-        // In shared context, .Random.seed is already initialized. Therefore, all these tests are ignored.
-        // TODO: Once the performance of non-shared context is sufficient, run these tests in non-shared context.
+        // In shared context, .Random.seed is already initialized. Therefore, all these tests are
+        // ignored.
+        // TODO: Once the performance of non-shared context is sufficient, run these tests in
+        // non-shared context.
         assertEval(Ignored.ImplementationError, Output.IgnoreErrorContext, "{ .Random.seed }");
         assertEval(Ignored.ImplementationError, Output.IgnoreErrorContext, "{ print(.Random.seed) }");
         assertEval(Ignored.ImplementationError, Output.IgnoreErrorContext, "{ get('.Random.seed', envir = .GlobalEnv, inherits = FALSE) }");

com.oracle.truffle.r.test/src/com/oracle/truffle/r/test/library/utils/TestInteractiveDebug.java

@@ -89,8 +89,8 @@ public void testPromise() {
     }
 
     /**
-     * FIXME: Ignored, because the shared context between unit tests does not handle debugging more functions at once
-     * correctly.
+     * FIXME: Ignored, because the shared context between unit tests does not handle debugging more
+     * functions at once correctly.
      */
     @Test
     @Ignore
@@ -114,7 +114,8 @@ public void testContinue() {
 
     @Test
     public void testDebugOnce() {
-        assertEval(TIMEOUT, Context.NonShared, Ignored.OutputFormatting, "fun0 <- function() { print('fun0') }; fun1 <- function() { print('en'); fun0(); fun0(); print('ex') }; debugonce(fun0); fun1()\nc\n");
+        assertEval(TIMEOUT, Context.NonShared, Ignored.OutputFormatting,
+                        "fun0 <- function() { print('fun0') }; fun1 <- function() { print('en'); fun0(); fun0(); print('ex') }; debugonce(fun0); fun1()\nc\n");
         assertEval(TIMEOUT, Context.NonShared, Ignored.OutputFormatting, "fun0 <- function() { print('fun0') }; debugonce(fun0); fun0()\nc\n");
     }
 
@@ -135,8 +136,10 @@ public void testBrowser() {
 
     @Test
     public void testSetBreakpoint() {
-        assertEval(TIMEOUT, Context.NonShared, Output.IgnoreDebugCallString, Output.IgnoreDebugPath, String.format("source('%s'); setBreakpoint('%s', 4, verbose=F); fun(10)\n\n\n\n\n\n\n\n", debugFile, debugFile));
-        assertEval(TIMEOUT, Context.NonShared, String.format("source('%s'); setBreakpoint('%s', 4, verbose=F); setBreakpoint('%s', 4, verbose=F, clear=T); fun(10)\n", debugFile, debugFile, debugFile));
+        assertEval(TIMEOUT, Context.NonShared, Output.IgnoreDebugCallString, Output.IgnoreDebugPath,
+                        String.format("source('%s'); setBreakpoint('%s', 4, verbose=F); fun(10)\n\n\n\n\n\n\n\n", debugFile, debugFile));
+        assertEval(TIMEOUT, Context.NonShared,
+                        String.format("source('%s'); setBreakpoint('%s', 4, verbose=F); setBreakpoint('%s', 4, verbose=F, clear=T); fun(10)\n", debugFile, debugFile, debugFile));
         assertEval(TIMEOUT, Context.NonShared, Output.IgnoreDebugCallString, Output.IgnoreDebugPath, String.format(
                         "source('%s'); setBreakpoint('%s', 4, verbose=F); fun(10)\n\n\n\n\n\n\n\n\nsetBreakpoint('%s', 4, verbose=F, clear=T); fun(10)\nsetBreakpoint('%s', 4, verbose=F); invisible(fun(10))\n\n\n\n\n\n\n\n\n",
                         debugFile, debugFile, debugFile, debugFile));

com.oracle.truffle.r.test/src/com/oracle/truffle/r/test/rng/TestRRNG.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2016, 2021, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2016, 2019, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -22,13 +22,11 @@
  */
 package com.oracle.truffle.r.test.rng;
 
-import org.junit.Ignore;
 import org.junit.Test;
 
 import com.oracle.truffle.r.test.TestBase;
 
 public class TestRRNG extends TestBase {
-    // FIXME: GR-35083
     @Test
     public void testDirectSeedAssignment() {
         // changes generator to MarsagliaMulticarry and sets its 2 seeds