Integrated: 8344841: ClassPrinter prints confusing value for null

Adam Sotona asotona at openjdk.org
Mon Nov 25 13:48:23 UTC 2024


On Fri, 22 Nov 2024 10:24:12 GMT, Adam Sotona <asotona at openjdk.org> wrote:

> ClassPrinter prints `null` values as `String` "null" or as `String.valueOf(ConstantDescs.BSM_NULL_CONSTANT)`, based on context.
> 
> This patch unifies handling of `null` values.
> They are now exclusively represented as `ConstantDescs.BSM_NULL_CONSTANT`.
> In Json and Yaml format is `null` value printed unquoted and in Xml as `<null/>`.
> In Json and Yaml is `"null"` `String` printed quoted to distinguish from the `null` value.
> 
> Please review.
> 
> Thank you,
> Adam

This pull request has now been integrated.

Changeset: c6b14c62
Author:    Adam Sotona <asotona at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/c6b14c62f609ee0539893fa3778eb5d0277748c0
Stats:     29 lines in 2 files changed: 17 ins; 0 del; 12 mod

8344841: ClassPrinter prints confusing value for null

Reviewed-by: liach

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

PR: https://git.openjdk.org/jdk/pull/22316


More information about the core-libs-dev mailing list