RFR: 8364103: Convert existing sprintf-chains to stringStream [v2]
Paul Hübner
phubner at openjdk.org
Fri Sep 5 14:03:56 UTC 2025
On Thu, 4 Sep 2025 17:49:36 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> Paul Hübner has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains three commits:
>>
>> - Incorporate reviewer feedback.
>> - Merge branch 'master' into JDK-8364103
>> - Refactor stack trace printing to use stringStream.
>
> Looks okay as is, but there is some optional tidying up that could be done.
Thanks for the suggestions, @kimbarrett. I've cleaned up as you suggested, which also gave way to fuse some of the conditionals, making it more readable overall. Could you take a look to double check no semantics were changed during the transformation?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/26894#issuecomment-3258456533
More information about the hotspot-runtime-dev
mailing list