Thank you for the review!
I'll make it consistent.
On 5/22/20 11:33, Chris Plummer wrote:
Just one very minor editing suggestion. Where you have "If
canUnrestrictedlyRedefineClasses() is false," there is a comma placed
here, but the previous two bullet items are of a similar form, yet
have no comma. I suggest you make all 3 consistent. I think either
with or without a comma is acceptable in this case. From what I read
if the opening phrase is 4 or fewer words, the comma is optional. I'm
just suggesting you be consistent.
On 5/21/20 10:02 PM, serguei.spit...@oracle.com wrote:
Please, review a fix for:
CSR draft (one CSR reviewer is needed before finalizing it):
The fix is to replace in Instrumentation, JDI and JDWP spec a
description of class
redefinition or retransformation restriction with a link to the
supporting JVM TI
function where it has been already documented.
This spec refactoring should help in cases when new 'unmodifiable
in redefinition' class file attributes are added.
Built docs and checked the link works as expected.