RFR (S) 8217994: os::print_hex_dump should be more resilient against unreadable memory
Schmidt, Lutz
lutz.schmidt at sap.com
Wed Jan 30 14:25:13 UTC 2019
Aleksey,
I like your revision 02. Very compact.
I do not like re-inventing language features by own code.
Sorry for spamming you with suggestions/comments.
Regards,
Lutz
On 30.01.19, 13:31, "Aleksey Shipilev" <shade at redhat.com> wrote:
On 1/30/19 1:10 PM, Thomas Stüfe wrote:
> Or, since this thing keeps coming up, add a helper to outputStream:
>
> +void outputStream::put(char ch, int repeat_count) {
> + for (int i = 0; i < repeat_count; i ++) {
> + put(ch);
> + }
> +}
> +
>
> st->put('?', 2*unitsize);
No. Let's stop here:
http://cr.openjdk.java.net/~shade/8217994/webrev.02/
There are thousand ways to do things, and there are only 8 work hours per day.
Current patch with unusual format specifier is already good, to my taste.
-Aleksey
More information about the hotspot-dev
mailing list