desruisseaux commented on code in PR #1086:
URL:
https://github.com/apache/maven-compiler-plugin/pull/1086#discussion_r3616049086
##########
src/main/java/org/apache/maven/plugin/compiler/AbstractCompilerMojo.java:
##########
@@ -649,6 +649,16 @@ final Charset charset() {
* then the default value is same as above with the addition of {@code
"rebuild-on-add,rebuild-on-change"}.
* It means that a full rebuild will be done if any kind of change is
detected.</p>
*
+ * <p>Whether an annotation processor is considered present depends on the
Java version when {@link #proc}
+ * is unset, because {@code javac} enables annotation processing by
default before Java 23
+ * ({@code -proc:full}) but disables it since Java 23 ({@code
-proc:none} unless a processor is
+ * configured). Consequently, on Java versions prior to 23 the
plugin conservatively assumes that
Review Comment:
Do we really need no break space (` `) before "Java" and "versions"
here?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
