RFR (trivial): 8229808: javaVFrame::print_lock_info_on fails to disable extra printing
david.holmes at oracle.com
Fri Aug 16 12:49:33 UTC 2019
I had to rebase now that Stefan pushed the markOop->markWord changes.
webrev updated in place.
On 16/08/2019 9:51 pm, Harold Seigel wrote:
> Looks good and trivial.
> Thanks, Harold
> On 8/16/2019 2:53 AM, David Holmes wrote:
>> 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.
More information about the hotspot-runtime-dev