Skip to content

Commit a5ab42a

Browse files
Apply clarification proposed by copilot.
Co-authored-by: Copilot <[email protected]>
1 parent 0b2c333 commit a5ab42a

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

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 for patching the module system.
671+
if (WorkaroundForPatchModule.ENABLED) {
671672
if (workaround instanceof WorkaroundForPatchModule wp) {
672673
workaround = wp.getFileManagerIfUsable();
673674
if (workaround == null) {

0 commit comments

Comments
 (0)