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