> [this is an improved version of openjdk/jdk PR:
> https://github.com/openjdk/jdk/pull/4428
> ]
> 
> The ct.sym may contain classfiles referring to annotations that are not 
> present in ct.sym (liek JDK's internal annotation sun.misc.Contended). If 
> javac will try to load them (while discovering annotations for the purpose of 
> detecting which annotation processors should be run), an error will be 
> produced (please see the issue). The proposal is to strip annotations that 
> are not present in ct.sym when generating ct.sym.
> 
> As noted by @jddarcy, we also need a special handling of ValueBased, which 
> this patch should do, so it also solves 
> https://bugs.openjdk.java.net/browse/JDK-8258421

Jan Lahoda has updated the pull request with a new target base due to a merge 
or a rebase. The incremental webrev excludes the unrelated changes brought in 
by the merge/rebase. The pull request contains six additional commits since the 
last revision:

 - Adding notes on annotations handled specially by javac, as suggested.
 - Merge branch 'master' into JDK-8266036
 - 8258421: (jdeprscan) tools/jdeprscan/tests/jdk/jdeprscan/TestRelease.java 
failed with "error: cannot access jdk.internal.ValueBased"
 - Removing a test from the ProblemList.txt
 - Merge branch 'master' into JDK-8266036
 - 8266035: class file for sun.misc.Contended not found

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk17/pull/27/files
  - new: https://git.openjdk.java.net/jdk17/pull/27/files/d9d44fde..187ad580

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk17&pr=27&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk17&pr=27&range=00-01

  Stats: 20125 lines in 373 files changed: 11802 ins; 7219 del; 1104 mod
  Patch: https://git.openjdk.java.net/jdk17/pull/27.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk17 pull/27/head:pull/27

PR: https://git.openjdk.java.net/jdk17/pull/27

Reply via email to