mbien commented on code in PR #7491:
URL: https://github.com/apache/netbeans/pull/7491#discussion_r1702001625


##########
java/java.editor.base/test/unit/src/org/netbeans/modules/java/editor/base/javadoc/JavadocImportsTest.java:
##########
@@ -207,6 +207,122 @@ public void testComputeReferencedElements() throws 
Exception {
         assertEquals(exp, sortedResult);
     }
     
+    public void testComputeReferencedElementsMarkdown() throws Exception {
+        String code =
+                "package p;\n" +
+                "import java.io.IOException;\n" +

Review Comment:
   sidenote: bumping the lang level for multi line Strings could make it easier 
to write new tests with code snippets.



##########
java/java.sourceui/src/org/netbeans/api/java/source/ui/ElementJavadoc.java:
##########
@@ -1909,6 +1965,9 @@ public SourceLineCharterMapperToHtmlTag(List<String> 
startTag, char sourceChar,
         }
     }
     private void appendReference(StringBuilder sb, ReferenceTree ref, List<? 
extends DocTree> label, TreePath docPath, DocCommentTree doc, DocTrees trees) {
+        if (ref == null) {
+            System.err.println("!!!!");
+        }

Review Comment:
   `System.err` used outside of tests



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists

Reply via email to