> `ValueBased` talks about constructor avoidance but does not specify the 
> reason. It's better to include the reason because it's not straightforward 
> from the fact that there's no binary incompatibility.
> 
> Also `InvalidClassException` benefits from descriptions about 
> strictly-initialized fields.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Chen Liang 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 three additional commits since 
the last revision:

 - Restructuring and rewording
 - Merge branch 'lworld' of https://github.com/openjdk/valhalla into 
doc/lw-vb-ctor-ice
 - Minor ValueBased and InvalidClassException comments

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/2631/files
  - new: https://git.openjdk.org/valhalla/pull/2631/files/fe7205a5..5a87b226

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

  Stats: 3921 lines in 190 files changed: 2596 ins; 618 del; 707 mod
  Patch: https://git.openjdk.org/valhalla/pull/2631.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2631/head:pull/2631

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

Reply via email to