> Seems like an oversight where "value" is added unconditionally. I changed the 
> behavior to add "identity" for preview identity classes, which was never 
> printed before (because identity is not a source modifier)
> 
> Testing: langtools/tools/javap + javac

Chen Liang has updated the pull request incrementally with one additional 
commit since the last revision:

  Don't emit spurious identity

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/1938/files
  - new: https://git.openjdk.org/valhalla/pull/1938/files/c06db063..50d7b76f

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=1938&range=01
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=1938&range=00-01

  Stats: 5 lines in 2 files changed: 1 ins; 0 del; 4 mod
  Patch: https://git.openjdk.org/valhalla/pull/1938.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1938/head:pull/1938

PR: https://git.openjdk.org/valhalla/pull/1938

Reply via email to