RFR (XS): 8196450: Deprecate JDWP/JDI canUnrestrictedlyRedefineClasses to match JVM TI capabilities

David Holmes david.holmes at oracle.com
Fri Jun 5 13:05:56 UTC 2020


Sorry Serguei I got distracted and forgot about the RFR part of this.

Reviewed :)

Thanks,
David

On 5/06/2020 3:19 pm, serguei.spitsyn at oracle.com wrote:
> Hi David,
> 
> You have already approved the CSR below.
> May I count it as a review as there is no difference between CSR and 
> webrev - both have the same spec update?
> 
> Thanks,
> Serguei
> 
> 
> On 6/3/20 20:57, serguei.spitsyn at oracle.com wrote:
>> Please, review a fix for:
>> https://bugs.openjdk.java.net/browse/JDK-8196450
>>
>>
>> CSR draft (one CSR reviewer is needed before finalizing it):
>> https://bugs.openjdk.java.net/browse/JDK-8246540
>>
>>
>> Webrev:
>> http://cr.openjdk.java.net/~sspitsyn/webrevs/2020/jdwp-depr.1/src/
>>
>>
>> Updated JDWP VirtualMachine::capabilitiesNew spec:
>> http://cr.openjdk.java.net/~sspitsyn/webrevs/2020/jdwp-depr.1/docs/specs/jdwp/jdwp-protocol.html#JDWP_VirtualMachine_CapabilitiesNew
>>
>> Updated JDI com.sun.jdi.VirtualMachine spec:
>> http://cr.openjdk.java.net/~sspitsyn/webrevs/2020/jdwp-depr.1/docs/api/jdk.jdi/com/sun/jdi/VirtualMachine.html#canAddMethod()
>> http://cr.openjdk.java.net/~sspitsyn/webrevs/2020/jdwp-depr.1/docs/api/jdk.jdi/com/sun/jdi/VirtualMachine.html#canUnrestrictedlyRedefineClasses()
>>
>>
>> Summary:
>>   The fix adds annotations and deprecation comments to the capabilities
>>    canUnrestrictedlyRedefineClasses and canAddMethod.
>>    It impacts the JDWP capabilitiesNew command and the JDI 
>> VirtualMachine interface.
>>
>>
>> Testing:
>>   Built docs and checked the doc has been generated as expected.
>>   Will run the JDI/JDWP tests locally
>>
>> Thanks,
>> Serguei
> 


More information about the serviceability-dev mailing list