RFR(S): 8171848: ObjectMonitor verify() and print() methods are empty

[Kirill Zhaldybin]

David,

Thank you for review!

Regards, Kirill

On 01.02.2017 07:47, David Holmes wrote:
> Hi Kirill,
>
> On 1/02/2017 2:11 AM, Kirill Zhaldybin wrote:
>> David,
>>
>> Here are a new WebRev:
>> http://cr.openjdk.java.net/~kzhaldyb/webrevs/JDK-8171848/webrev.01/
>> I deleted ObjectSynchronizer::verify too since I failed to find any
>> calls to it.
>>
>> Could you please let me know your opinion?
>
> Looks fine.
>
> Thanks,
> David
> -----
>
>> Thank you.
>>
>> Regards, Kirill
>>
>> On 21.12.2016 16:24, Kirill Zhaldybin wrote:
>>> David,
>>>
>>> Thank you for review!
>>>
>>> On 21.12.2016 15:46, David Holmes wrote:
>>>> Hi Kirill,
>>>>
>>>> This cleanup is considered an enhancement and as such is no longer
>>>> acceptable in JDK 9 - sorry. The CR has been re-targetted to JDK 10.
>>>>
>>>> The removal itself seems okay. I wonder whether
>>>> ObjectSynchronizer::verify is itself ever called?
>>> I did not find any calls to it but it was after I sent WebRev.
>>> I will double-check and will send new one if we can just delete
>>> ObjectSynchronizer::verify.
>>>
>>> Regards, Kirill
>>>>
>>>> David
>>>>
>>>> On 21/12/2016 8:36 PM, Kirill Zhaldybin wrote:
>>>>> Dear all,
>>>>>
>>>>> Could you please review this fix for 8171848?
>>>>> I deleted both methods from ObjectMonitor and the code which was not
>>>>> doing anything from ObjectSynchronizer::verify().
>>>>> I did local and rbt testing to be sure that I found all calls to
>>>>> deleted
>>>>> methods.
>>>>>
>>>>> WebRev:
>>>>> http://cr.openjdk.java.net/~kzhaldyb/webrevs/JDK-8171848/webrev.00/
>>>>> CR: https://bugs.openjdk.java.net/browse/JDK-8171848
>>>>>
>>>>> Thank you.
>>>>>
>>>>> Regards, Kirill
>>>
>>