RFR: 8262491: AArch64: CPU description should contain compatible board list [v6]

Yasumasa Suenaga ysuenaga at openjdk.java.net
Wed Mar 10 13:00:08 UTC 2021


On Wed, 3 Mar 2021 16:49:39 GMT, Andrew Haley <aph at openjdk.org> wrote:

>> So I set `\0` to the tail of `buf` at L178.
>
>> So I set `\0` to the tail of `buf` at L178.
> 
> Which you then overwrite on Line 183. You really do need to move Line 174 to the end.
> I ran a test with the read fulling the whole buffer.

Thanks for your comment! I pushed new commit.

I issue `read()` with `buflen - 1` bytes, then I set `\0' to tail of `buf` after `read()`. And also `\0` will be converted to blank will happen when `read()` return 1 or greater. How about it?

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

PR: https://git.openjdk.java.net/jdk/pull/2759


More information about the hotspot-dev mailing list