RFR: 8344841: ClassPrinter prints confusing value for null [v2]
Chen Liang
liach at openjdk.org
Mon Nov 25 13:34:24 UTC 2024
On Mon, 25 Nov 2024 07:35:58 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
>
> Adam Sotona has updated the pull request incrementally with one additional commit since the last revision:
>
> Simplified mapping
Marked as reviewed by liach (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/22316#pullrequestreview-2458448344
More information about the core-libs-dev
mailing list