Jan Lahoda

Results 98 comments of Jan Lahoda

Unless there are objections, I'll integrate once/if tests will pass.

> I saw this exception twice in CI logs of this PR: https://github.com/apache/netbeans/actions/runs/9366885368/attempts/1 https://github.com/apache/netbeans/actions/runs/9319212924/attempts/1 > > I don't remember this being a common failure. (CV tests on linux typically don't...

FWIW, does not handle static imports quite correctly, looking into a fix.

FWIW, I've opened: https://github.com/JaroslavTulach/nb-javac/pull/30

> `ComputeImportsTest` failed due to CCE in javac Lint class. Likely due to [openjdk/jdk#22056](https://github.com/openjdk/jdk/pull/22056), disabled test case since it influenced code scanning of other tests. Thanks. FWIW, I've opened: https://github.com/openjdk/jdk/pull/26142...

> noticed during code scanner smoke test: > > ```java > public static void main(String[] args) { > new WeakReference(null) { > private Object hardRef = null; > }; >...

> goodbye string templates, lets hope they will be back at some point in future. > > This is something we should mention on the download page since it might...

The plan is to finalize with a new (currently up-to-date) javac. Ideally, sometime during this week, I think. Thanks for the comments!

The nb-javac binary download has been updated to JDK 23 build 30, as uploaded to Maven central, I think I've fixed all the comments so far. So I think this...