RFR (trivial): 8229808: javaVFrame::print_lock_info_on fails to disable extra printing

David Holmes david.holmes at oracle.com
Fri Aug 16 06:53:10 UTC 2019


bug: https://bugs.openjdk.java.net/browse/JDK-8229808
webrev: http://cr.openjdk.java.net/~dholmes/8229808/webrev/

Trivial code cleanup that Stefan noticed. The original code with 
disabled printing was added in JDK-8069412 using the Verbose flag 
combined with a non-NULL mark. The use of the verbose flag was replaced 
by ObjectMonitor::Knob_verbose in JDK-8130448. Then the printing code 
was removed by JDK-8210848 "Obsolete SyncKnobs" - at which point the 
comment about "disable extra printing below" should also have been 
removed, and the mark variable returned to being local to the if-clause. 
There's also an earlier comment relating to lockbits that refers to the 
printing code that was deleted, so its gone too.

Thanks,
David





More information about the hotspot-runtime-dev mailing list