RFR: 8332497: javac prints an AssertionError when annotation processing runs on program with module imports [v14]
Evemose
duke at openjdk.org
Wed May 22 20:33:33 UTC 2024
> Fix is pretty simple: visitModuleImport in com.sun.tools.javac.tree.TreeScanner has notbeen overriden, so defaulted to Visitor::visitModuleImport, which forwards to Visitor::visitTree, which is also not overriden, and, therefore, threw AssertionError.
>
> PS: Im not even sure how it worked before without crashing, seems like there is some intermidiate implementation between this TreeScanner and actual scanners because otherwise it should have resultedin compile error the moment it encounter module importin any visitor
Evemose has updated the pull request incrementally with two additional commits since the last revision:
- Update ModuleImportProcessingTest.java
removed --enable-preview from @compile/process
- Replaced @build with @compile
Co-authored-by: liach <7806504+liach at users.noreply.github.com>
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/19292/files
- new: https://git.openjdk.org/jdk/pull/19292/files/0cfe4ac8..64280b42
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=19292&range=13
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=19292&range=12-13
Stats: 3 lines in 1 file changed: 0 ins; 0 del; 3 mod
Patch: https://git.openjdk.org/jdk/pull/19292.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/19292/head:pull/19292
PR: https://git.openjdk.org/jdk/pull/19292
More information about the compiler-dev
mailing list