desruisseaux commented on code in PR #1027:
URL:
https://github.com/apache/maven-compiler-plugin/pull/1027#discussion_r2753572194
##########
src/main/java/org/apache/maven/plugin/compiler/ToolExecutor.java:
##########
@@ -72,11 +73,6 @@
* @author Martin Desruisseaux
*/
public class ToolExecutor {
- /**
- * The locale for diagnostics, or {@code null} for the platform default.
- */
- private static final Locale LOCALE = null;
-
Review Comment:
While this constant is null for now, its purpose is to make easier to find
locale-sensitive codes in the future if we want to take locale in account. It
may not happen in near term, but if it happens in the future, I would search
for `Locale` or `LOCALE` on the whole code base.
--
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]
