Hi Alex,

The change looks good to me. 

Thanks!

-Daniil

On 9/6/19, 12:59 PM, "serviceability-dev on behalf of Alex Menkov" 
<[email protected] on behalf of 
[email protected]> wrote:

    Hi all,
    
    Please review the fix for
       https://bugs.openjdk.java.net/browse/JDK-8230516
    webrev:
     
    http://cr.openjdk.java.net/~amenkov/jdk14/jdwp-protocol_table_span/webrev/
    
    The fix moves "anchor" spans to inside the cells.
    
    generated docs (no visual changes):
    old:
     
    
http://cr.openjdk.java.net/~amenkov/jdk14/jdwp-protocol_table_span/0/jdwp-protocol.html
    new:
     
    
http://cr.openjdk.java.net/~amenkov/jdk14/jdwp-protocol_table_span/1/jdwp-protocol.html
    
    --alex
    
    


Reply via email to