Jan Lahoda

Results 98 comments of Jan Lahoda

Thanks! I'll merge if there's a clean test run.

Thanks for the comments so far! Here I tried to reflect them: https://github.com/openjdk/jdk/pull/21431/commits/b7c871dab8357864b532a1f217ed26c3b40d50f4 (For `ClassFileVersionsTest.java`, it seemed unnecessarily complex to position the test cases into specific array positions, so I...

FWIW: I've changed the implementation to an explicit priority "queue", as the original patch with injecting tasks in the editor background infrastructure proved to be too complex, at least for...

I limitation of this way to cancel things is that it cannot stop computations inside javac/parser. Unlike the previous round which used the "editor" tasks, which can stop even javac/parser....

@matthiasblaesing, uh. You are right, of course. Thanks for diagnosing it so deeply. I've filled: https://bugs.openjdk.org/browse/JDK-8333107

Just, fwiw, my understanding (which may be wrong, of course) is that the indexing depends on some Lucene 3.X feature(s), which no longer exists in the newer version. So, what...

I think this will be a good cleanup.