> On Apr 5, 2016, at 4:48 PM, Brent Christian <brent.christ...@oracle.com> 
> wrote:
> 
> Thanks, Coleen.  Coordinating method/function names on "to stack trace 
> element" is a fine thing.  I've done so in the updated webrev, and also 
> implemented Claes's suggestion.
> 
> http://cr.openjdk.java.net/~bchristi/8153123/webrev.01/index.html
> 

Looks good.  Nit: can you add a space after “synchronized” in 
StackFrameInfo.java line 109:
 109             synchronized(this) {

Mandy

Reply via email to