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


##########
java/spi.java.hints/src/org/netbeans/modules/java/hints/spiimpl/batch/BatchSearch.java:
##########
@@ -246,8 +247,15 @@ public void run(CompilationController parameter) throws 
Exception {
                                 boolean cont = true;
 
                                 try {
-                                    if 
(parameter.toPhase(Phase.RESOLVED).compareTo(Phase.RESOLVED) < 0)
+                                    if 
(parameter.toPhase(Phase.RESOLVED).compareTo(Phase.RESOLVED) < 0) {
+                                        if (currentInputList.size() == 1) {
+                                            //the javac crashed while 
processing the (single) file, we must ensure progress, otherwise infinite loop 
in processing would happen:
+                                            problems.add(new 
MessageImpl(MessageKind.WARNING, "An error occurred while processing file: " + 
FileUtil.getFileDisplayName(parameter.getFileObject()) + ", please see the IDE 
log for more information."));
+                                            currentPointer.incrementAndGet();
+                                        }
+
                                         return ;
+                                    }

Review Comment:
   sorry never mind this, I just noticed Jan posted a long explanation as 
comment on the 
[issue](https://github.com/apache/netbeans/issues/4054#issuecomment-1407651558) 
which answers the questions. Although the retry counter might be still 
something we could consider to ensure there is no scenario where this loop 
can't exit.



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