Minor clean up for processing#384.

sampottinger · sampottinger · commit 077e6741f72e · 2022-01-29T20:21:22.000-08:00
diff --git a/java/src/processing/mode/java/PreprocService.java b/java/src/processing/mode/java/PreprocService.java
@@ -957,7 +957,7 @@ private static boolean checkIfImportsChanged(List<ImportStatement> prevImports,
   static {
     Map<String, String> compilerOptions = new HashMap<>();
 
-    // TODO: VERSION_17 if using new language features
+    // TODO: VERSION_17 if using new language features. Requires new JDT.
     compilerOptions.put(JavaCore.COMPILER_COMPLIANCE, JavaCore.VERSION_11);
     compilerOptions.put(JavaCore.COMPILER_SOURCE, JavaCore.VERSION_11);
     compilerOptions.put(JavaCore.COMPILER_CODEGEN_TARGET_PLATFORM, JavaCore.VERSION_11);
diff --git a/java/src/processing/mode/java/SourceUtil.java b/java/src/processing/mode/java/SourceUtil.java
@@ -330,7 +330,7 @@ static public void scrubCommentsAndStrings(StringBuilder p) {
         } else {
           // Exiting block
           int blockEnd = i;
-          if (prevState == IN_BLOCK_COMMENT && i < length) blockEnd--;
+          if (prevState == IN_BLOCK_COMMENT && i < length) blockEnd--;  // preserve star in '*/'
           for (int j = blockStart; j < blockEnd; j++) {
             char c = p.charAt(j);
             if (c != '\n' && c != '\r') p.setCharAt(j, ' ');
diff --git a/java/src/processing/mode/java/preproc/PdeParseTreeListener.java b/java/src/processing/mode/java/preproc/PdeParseTreeListener.java
@@ -317,7 +317,15 @@ public void exitProcessingSketch(ProcessingParser.ProcessingSketchContext ctx) {
     footerResult = prepareFooter(rewriter, length);
   }
 
-  @Override
+  /**
+   * Detect if the user is programming with "mixed" modes.
+   *
+   * <p>Detect if the user is programming with "mixed" modes where they are
+   * combining active and static mode features. This may be, for example, a
+   * method call followed by method definition.</p>
+   *
+   * @param ctx The context from ANTLR for the mixed modes sketch.
+   */
   public void enterWarnMixedModes(ProcessingParser.WarnMixedModesContext ctx) {
     pdeParseTreeErrorListenerMaybe.ifPresent((listener) -> {
       Token token = ctx.getStart();
