RFR: 8344079: Minor fixes and cleanups to compiler lint-related code [v4]
Archie Cobbs
acobbs at openjdk.org
Mon Dec 2 14:20:47 UTC 2024
On Mon, 2 Dec 2024 13:38:23 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>> src/jdk.compiler/share/classes/com/sun/tools/javac/processing/JavacProcessingEnvironment.java line 717:
>>
>>> 715: processor, log, lint));
>>> 716: if (lint && !patternAdded) {
>>> 717: log.warning(PROCESSING, null, Warnings.ProcDuplicateSupportedAnnotation(annotationPattern,
>>
>> Did you run all langtools tests? I believe that if you change these, some test output will be broken, as an extra `[processing]` will be emitted. I'm seeing this while playing in a related area.
>
> Example: test/langtools/tools/doclint/AnonClassTest.java
> Did you run all langtools tests?
I did not run any tests beyond those that are run by GA - but they are all passing.
> Example: test/langtools/tools/doclint/AnonClassTest.java
When I run that test manually it passes... are you seeing it fail?
$ make test TEST="test/langtools/tools/doclint/AnonClassTest.java"
Building target 'test' in configuration 'macosx-aarch64-server-release'
Test selection 'test/langtools/tools/doclint/AnonClassTest.java', will run:
* jtreg:test/langtools/tools/doclint/AnonClassTest.java
Running test 'jtreg:test/langtools/tools/doclint/AnonClassTest.java'
Passed: tools/doclint/AnonClassTest.java
Test results: passed: 1
Report written to /Users/archie/proj/jdk/build/macosx-aarch64-server-release/test-results/jtreg_test_langtools_tools_doclint_AnonClassTest_java/html/report.html
Results written to /Users/archie/proj/jdk/build/macosx-aarch64-server-release/test-support/jtreg_test_langtools_tools_doclint_AnonClassTest_java
Finished running test 'jtreg:test/langtools/tools/doclint/AnonClassTest.java'
Test report is stored in build/macosx-aarch64-server-release/test-results/jtreg_test_langtools_tools_doclint_AnonClassTest_java
==============================
Test summary
==============================
TEST TOTAL PASS FAIL ERROR
jtreg:test/langtools/tools/doclint/AnonClassTest.java
1 1 0 0
==============================
TEST SUCCESS
Finished building target 'test' in configuration 'macosx-aarch64-server-release'
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22056#discussion_r1865930657
More information about the compiler-dev
mailing list