RFR: 8333211: NMT Reports: replace manual indentation handling with auto indent [v2]

Thomas Stuefe stuefe at openjdk.org
Tue Jun 4 11:47:29 UTC 2024


On Tue, 4 Jun 2024 11:14:51 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

>> Thomas Stuefe has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>> 
>>  - restore indentation() accessor
>>  - Merge branch 'master' into nmt-reporting-better-indentation
>>  - NMT reporting, better indentation
>
> Nice!
> 
> Maybe keep `indentation`/`set_indentation`, they can be removed in a separate PR. Let's us to not have to wait on the PR that David mentioned.

Thanks @jdksjolen ! I restored the indentation accessors.

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

PR Comment: https://git.openjdk.org/jdk/pull/19461#issuecomment-2147325157


More information about the hotspot-runtime-dev mailing list