Skip to content

Commit f462016

Browse files
desruisseauxCopilot
andcommitted
Fix spelling errors and apply clarification proposed by copilot.
Co-authored-by: Copilot <[email protected]>
1 parent cbb7e89 commit f462016

File tree

5 files changed

+6
-5
lines changed

5 files changed

+6
-5
lines changed

src/it/multirelease-on-classpath/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
<artifactId>multirelease-on-classpath</artifactId>
2424
<version>1.0-SNAPSHOT</version>
2525
<packaging>jar</packaging>
26-
<name>Mulirelease in Maven 4</name>
26+
<name>Multirelease in Maven 4</name>
2727

2828
<build>
2929
<plugins>

src/it/multirelease-with-modules/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
<artifactId>multirelease-with-modules</artifactId>
2424
<version>1.0-SNAPSHOT</version>
2525
<packaging>jar</packaging>
26-
<name>Mulirelease with modules</name>
26+
<name>Multirelease with modules</name>
2727

2828
<build>
2929
<plugins>

src/main/java/org/apache/maven/plugin/compiler/SourcesForRelease.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ final class SourcesForRelease implements Closeable {
8383
Map<PathType, List<Path>> dependencySnapshot;
8484

8585
/**
86-
* The output directory for the release. This is either base base output directory or a sub-directory
86+
* The output directory for the release. This is either the base output directory or a sub-directory
8787
* in {@code META-INF/versions/}. This field is not used by this class, but made available for making
8888
* easier to write the {@code target/javac.args} debug file.
8989
*/

src/main/java/org/apache/maven/plugin/compiler/ToolExecutor.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -667,7 +667,8 @@ public boolean compile(final JavaCompiler compiler, final Options configuration,
667667
Iterable<? extends JavaFileObject> sources = fileManager.getJavaFileObjectsFromPaths(c.files);
668668
StandardJavaFileManager workaround = fileManager;
669669
boolean workaroundNeedsClose = false;
670-
if (WorkaroundForPatchModule.ENABLED) { // Test alone for making clear everything inside is a hack.
670+
// Check flag separately to clearly indicate this entire block is a workaround hack.
671+
if (WorkaroundForPatchModule.ENABLED) {
671672
if (workaround instanceof WorkaroundForPatchModule wp) {
672673
workaround = wp.getFileManagerIfUsable();
673674
if (workaround == null) {

src/main/java/org/apache/maven/plugin/compiler/WorkaroundForPatchModule.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
* Workaround for a {@code javax.tools} method which seems not yet supported on all compilers.
4242
* At least with OpenJDK 24, an {@link UnsupportedOperationException} may occur during the call to
4343
* {@code fileManager.setLocationForModule(StandardLocation.PATCH_MODULE_PATH, moduleName, paths)}.
44-
* The workaround is to format the paths in a {@code --path-module} option instead.
44+
* The workaround is to format the paths in a {@code --patch-module} option instead.
4545
* The problem is that we can specify this option only once per file manager instance.
4646
*
4747
* <p>We may remove this workaround in a future version of the Maven Compiler Plugin

0 commit comments

Comments
 (0)