Fix spelling typos and grammar errors.

Author: ascopes

java-compiler-testing/src/main/java/io/github/ascopes/jct/assertions/AbstractJavaFileObjectAssert.java

@@ -168,10 +168,10 @@ public JavaFileObjectKindAssert kind() {
 
   private byte[] rawContent() {
     return uncheckedIo(() -> {
-      var baos = new ByteArrayOutputStream();
-      try (var is = actual.openInputStream()) {
-        is.transferTo(baos);
-        return baos.toByteArray();
+      var outputStream = new ByteArrayOutputStream();
+      try (var inputStream = actual.openInputStream()) {
+        inputStream.transferTo(outputStream);
+        return outputStream.toByteArray();
       }
     });
   }

java-compiler-testing/src/main/java/io/github/ascopes/jct/assertions/PackageContainerGroupAssert.java

@@ -248,7 +248,7 @@ private <T> String fuzzySafePath(Iterable<T> parts) {
   }
 
   private boolean fileNameIsPresent(@Nullable Path path) {
-    // Path can be null if no path elements exist in ZipPath impls.
+    // Path can be null if no path elements exist in ZipPath implementations.
     return Optional
         .ofNullable(path)
         .map(Path::getFileName)

java-compiler-testing/src/main/java/io/github/ascopes/jct/compilers/JctCompiler.java

@@ -430,7 +430,7 @@ default JctCompiler addCompilerOptions(String compilerOption, String... compiler
    * {@link #DEFAULT_PREVIEW_FEATURES}.
    *
    * <p>Generally, this feature should be avoided if testing across multiple versions
-   * of Java, as preview features are oftenvnot finalised and may change without warning.
+   * of Java, as preview features are often not finalised and may change without warning.
    *
    * @param enabled {@code true} to enable preview features, or {@code false} to disable them.
    * @return this compiler object for further call chaining.
@@ -650,7 +650,7 @@ default JctCompiler release(SourceVersion release) {
    *
    * @return this compiler object for further call chaining.
    * @throws UnsupportedOperationException if the current JVM version does not correspond to a
-   *                                       supported Jave release version in the compiler, or if the
+   *                                       supported Java release version in the compiler, or if the
    *                                       compiler does not support integral version numbers.
    * @since 1.1.0
    */

java-compiler-testing/src/main/java/io/github/ascopes/jct/compilers/impl/JctCompilationFactoryImpl.java

@@ -56,7 +56,7 @@ public final class JctCompilationFactoryImpl implements JctCompilationFactory {
 
   private static final Logger log = LoggerFactory.getLogger(JctCompilationFactoryImpl.class);
   private static final String ROOT_PACKAGE = "";
-  
+
   private final JctCompiler compiler;
 
   public JctCompilationFactoryImpl(JctCompiler compiler) {
@@ -132,7 +132,7 @@ public JctCompilation createCompilation(
           .addArgument(() -> success ? "completed successfully" : "failed")
           .addArgument(compilationExecutionTimeMs)
           .addArgument(() -> String.format(
-              "%.2f", 
+              "%.2f",
               (1000.0 * compilationUnits.size()) / compilationExecutionTimeMs
           ))
           .log();
@@ -219,7 +219,7 @@ private Collection<JavaFileObject> filterCompilationUnitsByBinaryNames(
     for (var className : classNames) {
       var compilationUnit = binaryNamesToCompilationUnits.get(className);
       if (compilationUnit == null) {
-        throw new JctCompilerException("No compilation unit matching " + className 
+        throw new JctCompilerException("No compilation unit matching " + className
             + " found in the provided sources");
       }
     }
@@ -239,9 +239,8 @@ private Collection<JavaFileObject> filterCompilationUnitsByBinaryNames(
   // interface does not expose this information consistently.
   // All JCT implementations should be using PathFileObject types internally
   // anyway, so this should be fine as a hack for now. In the future I may decide
-  // to add an additional set of methods to PathFileObject to expose searching
-  // for PathFileObjects directly to prevent the cast back to JavaFileObject that
-  // makes us need this hack.
+  // to add a set of methods to PathFileObject to expose searching for PathFileObjects
+  // directly to prevent the cast back to JavaFileObject that makes us need this hack.
   private PathFileObject forceUpcastJavaFileObject(JavaFileObject jfo) {
     return (PathFileObject) jfo;
   }

java-compiler-testing/src/main/java/io/github/ascopes/jct/compilers/impl/JctCompilationImpl.java

@@ -216,7 +216,7 @@ public Builder diagnostics(
     /**
      * Set the file manager.
      *
-     * <p>The file manager will not be closed once finshed with.
+     * <p>The file manager will not be closed once finished with.
      *
      * @param fileManager the file manager.
      * @return this builder.

java-compiler-testing/src/main/java/io/github/ascopes/jct/containers/PackageContainerGroup.java

@@ -138,10 +138,10 @@ public interface PackageContainerGroup extends ContainerGroup {
    *   // to achieve the same outcome with simpler syntax.
    *
    *   ClassLoader cl = containerGroup.getClassLoader();
-   *   try (InputStream is = cl.getResourceAsStream("META-INF/spring.factories")) {
-   *     ByteArrayOutputStream baos = new ByteArrayOutputStream();
-   *     is.transferTo(baos);
-   *     String content = new String(baos.toByteArray(), ...);
+   *   try (InputStream inputStream = cl.getResourceAsStream("META-INF/spring.factories")) {
+   *     ByteArrayOutputStream outputStream = new ByteArrayOutputStream();
+   *     inputStream.transferTo(outputStream);
+   *     String content = new String(outputStream.toByteArray(), ...);
    *     ...
    *   }
    * </code></pre>

java-compiler-testing/src/main/java/io/github/ascopes/jct/containers/impl/AbstractPackageContainerGroup.java

@@ -108,7 +108,7 @@ public void addPackage(PathRoot path) {
     var actualPath = path.getPath();
 
     // Null filename implies the path is the root directory of a file system (
-    // like a JIMFS RAM file system we initialize elsewhere).
+    // like a MemoryFileSystem RAM file system we initialize elsewhere).
     var isArchive = actualPath.getFileName() != null && ARCHIVE_EXTENSIONS
         .stream()
         .anyMatch(actualPath.getFileName().toString().toLowerCase(Locale.ROOT)::endsWith);

java-compiler-testing/src/main/java/io/github/ascopes/jct/filemanagers/config/JctFileManagerConfigurerChain.java

@@ -45,7 +45,7 @@
  * This enables configurers to wrap file managers in proxies or delegating implementations
  * to intercept or override existing behaviours.
  *
- * <p>This class is not threadsafe.
+ * <p>This class is not thread-safe.
  *
  * @author Ashley Scopes
  * @since 0.0.1

java-compiler-testing/src/main/java/io/github/ascopes/jct/utils/StringSlicer.java

@@ -25,7 +25,7 @@
 /**
  * Utility to efficiently split strings using a delimiter without using regular expressions.
  *
- * <p>This implementation is stateless and threadsafe.
+ * <p>This implementation is stateless and thread safe.
  *
  * @author Ashley Scopes
  * @since 0.0.1

java-compiler-testing/src/main/java/io/github/ascopes/jct/utils/StringUtils.java

@@ -61,7 +61,7 @@ private StringUtils() {
    * <p>This is designed to be able to take an input such as {@code List.of("foo", "bar", "baz")},
    * and be able to produce a string result such as {@code "foo, bar, or baz"} (where
    * {@code connector} in this case would be {@code ", "} and {@code lastConnector} would be
-   * {@code ", or "}.
+   * {@code ", or "}).
    *
    * <p>If no arguments are available, then an empty string is output instead.
    *