RFR: 8256828: ostream::print_cr() truncates buffer in copy-through case [v4]

Thomas Stuefe stuefe at openjdk.org
Tue Jun 4 07:45:02 UTC 2024


On Tue, 4 Jun 2024 07:34:34 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Clarifies the behaviour of this function in regards to truncation when adding a CR. Ensures a truncation warning is always issued.
>> 
>> Adds unit testing for the specialized cases.
>> 
>> See JBS for discussion of other suggestions.
>> 
>> Testing: - tiers 1-4
>> 
>> Thanks
>
> David Holmes has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Simplified the test code - thanks @tstuefe!
>   Rewrote the comment block describing do_vsnprintf.

Good. Thank you for taking this on.

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

Marked as reviewed by stuefe (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/19512#pullrequestreview-2095579155


More information about the hotspot-dev mailing list