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