> This PR is adding a new serialization warning for the case when a value class 
> declares one of the following serialization related methods:
> 
> - readObject
> - writeObject
> - readObjectNoData
> 
> They have no effect if declared in value classes, fact that should be 
> notified to the user,
> 
> TIA

Vicente Romero 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 16 additional commits 
since the last revision:

 - Merge branch 'lworld' into JDK-8378908
 - Merge branch 'JDK-8378908' of 
https://github.com/vicente-romero-oracle/valhalla into JDK-8378908
 - Apply suggestions from code review
   
   Co-authored-by: Chen Liang <[email protected]>
 - Apply suggestion from @liach
   
   Co-authored-by: Chen Liang <[email protected]>
 - Apply suggestion from @liach
   
   Co-authored-by: Chen Liang <[email protected]>
 - addressing review comments
 - adding more documentation
 - Merge branch 'lworld' into JDK-8378908
 - doc change
 - addressing review comments
 - ... and 6 more: https://git.openjdk.org/valhalla/compare/bb4c2406...a69dbab0

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/2187/files
  - new: https://git.openjdk.org/valhalla/pull/2187/files/3630bfbe..a69dbab0

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2187&range=08
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2187&range=07-08

  Stats: 19869 lines in 727 files changed: 9647 ins; 4309 del; 5913 mod
  Patch: https://git.openjdk.org/valhalla/pull/2187.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2187/head:pull/2187

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

Reply via email to