On Fri, 22 Aug 2025 16:50:53 GMT, Vicente Romero <[email protected]> wrote:
>> Vicente Romero has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 17 commits: >> >> - fixing bugs: new test cases brought in with latest merge are not accepted >> - merge with lworld >> - refactorings >> - another bug fix >> - bug fix >> - merge with lworld >> - renaming methods in LocalProxyVarGen >> - more changes >> - refactoring >> - test changes >> - ... and 7 more: >> https://git.openjdk.org/valhalla/compare/62e90b5d...0eb81681 > > src/jdk.compiler/share/classes/com/sun/tools/javac/comp/LocalProxyVarsGen.java > line 137: > >> 135: .filter(t -> t.hasTag(VARDEF)) >> 136: .map(t -> (JCVariableDecl)t) >> 137: .filter(vd -> (vd.sym.isStrict() || >> vd.sym.isFinal()) && !vd.sym.isStatic()) > > semantic change doing some changes here, I realized there is a bug in the PR, working on it now ------------- PR Review Comment: https://git.openjdk.org/valhalla/pull/1523#discussion_r2298029304
