RFR: 8299635: Hotspot update for deprecated sprintf in Xcode 14 [v5]

Xue-Lei Andrew Fan xuelei at openjdk.org
Wed Jan 18 20:07:35 UTC 2023


On Wed, 18 Jan 2023 19:45:55 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> I can see both styles are used in this file.  The current code here uses 8 whitespaces indent, I would like to keep it.
>
> I scanned the first 1000 lines and saw many cases of multi-line argument passing, and they all indent as David suggested. I'm not saying there are non in the file that indent has you have, but it seems if there are cases, they are the exception.

OK.  I have no strong preference.  Updated to align with the 1st parameter.

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

PR: https://git.openjdk.org/jdk/pull/11935


More information about the serviceability-dev mailing list