Integrated: 8339030: frame::print_value_on(outputStream* st, JavaThread *thread) doesn't need thread argument

Leonid Mesnik lmesnik at openjdk.org
Wed Aug 28 20:20:23 UTC 2024


On Tue, 27 Aug 2024 00:19:04 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

This pull request has now been integrated.

Changeset: d03ec7aa
Author:    Leonid Mesnik <lmesnik at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/d03ec7aad41d830b47801b7af75ee5e278128e69
Stats:     16 lines in 7 files changed: 0 ins; 0 del; 16 mod

8339030: frame::print_value_on(outputStream* st, JavaThread *thread) doesn't need thread argument

Reviewed-by: dholmes, coleenp

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

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


More information about the hotspot-dev mailing list