RFR: 8293251: Use stringStream::base() instead of as_string() when applicable [v7]
Robbin Ehn
rehn at openjdk.org
Thu Sep 15 11:49:43 UTC 2022
On Wed, 14 Sep 2022 13:32:46 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:
>> Hi!
>>
>> Please review this PR swapping out stringStream::as_string() with ::base() when applicable. With this change we avoid allocating a resource managed string copy. I also attempt to document the base function, as it is not safe to use any result returned from it if you subsequently call the stringStream's methods.
>>
>> When I've left ResourceMarks in place I've also commented which calls requires them.
>>
>> This passes tier1, tier2 tests.
>
> Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision:
>
> Rename to freeze
Seems fine, thanks.
-------------
Marked as reviewed by rehn (Reviewer).
PR: https://git.openjdk.org/jdk/pull/10142
More information about the hotspot-dev
mailing list