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

Leonid Mesnik lmesnik at openjdk.org
Tue Aug 27 17:14:38 UTC 2024


> 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

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/20721/files
  - new: https://git.openjdk.org/jdk/pull/20721/files/9e0304da..cd2f8192

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=20721&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=20721&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/20721.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/20721/head:pull/20721

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


More information about the hotspot-dev mailing list