> 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