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