RFR: 8346990: Remove INTX_FORMAT and UINTX_FORMAT macros [v2]

Coleen Phillimore coleenp at openjdk.org
Tue Jan 7 12:51:33 UTC 2025


On Tue, 7 Jan 2025 08:48:08 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> I don't think we should be mixing uintx types and UINTPTR_FORMAT like that.  As I said earlier, this is one that
>> I think probably ought not be changed at all.  I think some of the FORMAT macros are useful to avoid inline
>> format directives that resemble line noise, or ugly conditionals like that.
>
> Improving on my prior suggestion
> `"%#.*zx", (2 * BytesPerWord), _secondary_supers_bitmap`
> Using precision rather than field width, to avoid needing to account for the prefix in the width calculation.
> But still looking a lot like line noise, and still think it shouldn't be changed.

Yes, this looks horrible.  The macro that I was trying to remove is better.  I restored but moved it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22916#discussion_r1905405696


More information about the serviceability-dev mailing list