On Tue, 21 Jul 2026 16:53:30 GMT, Chen Liang <[email protected]> wrote:
> The Deserializer documentation states that it only applies to the boot class > path: > > https://github.com/openjdk/valhalla/blob/1c3301cb96d779d06bb6e2391c35ce729bc28189/src/java.base/share/classes/jdk/internal/value/Deserializer.java#L58-L59 > > > There is an implementation defect right now that this annotation is scanned > from all sources. > > Updated `ValueSerializationTest` to add positive and negative tests for a > value class with `@Deserializer` both on and off the boot class path. > > In the test updates I noticed `SerializedObjectCombo` is broken that it never > generates value classes somehow; tracked in > [JDK-8388597](https://bugs.openjdk.org/browse/JDK-8388597). > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). This pull request has now been integrated. Changeset: ff799ce4 Author: Chen Liang <[email protected]> URL: https://git.openjdk.org/valhalla/commit/ff799ce4a4e007220f999b9ce0d24a618e0917a8 Stats: 35 lines in 3 files changed: 30 ins; 0 del; 5 mod 8388318: [lworld] jdk.internal.value.Deserializer annotation should not be honored on application classes Reviewed-by: dlsmith, alanb, jpai ------------- PR: https://git.openjdk.org/valhalla/pull/2661
