RFR: 8339030: frame::print_value_on(outputStream* st, JavaThread *thread) doesn't need thread argument [v2]
David Holmes
dholmes at openjdk.org
Wed Aug 28 02:09:18 UTC 2024
On Tue, 27 Aug 2024 17:14:38 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:
>> Method
>> frame::print_value_on(outputStream* st, JavaThread *thread) doesn't need thread argument
>>
>> it usually is called with nullptr as second arg except
>> JavaThread::trace_frames()
>> where it is called with this.
>>
>> It seems that thread has never been used since 2007 so makes sense just to get rid of it.
>>
>> Tested building all builds available in CI and running tier13
>
> Leonid Mesnik has updated the pull request incrementally with one additional commit since the last revision:
>
> fixed identation
Marked as reviewed by dholmes (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/20721#pullrequestreview-2264845506
More information about the hotspot-dev
mailing list