> On Apr 5, 2016, at 4:48 PM, Brent Christian <[email protected]> > 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
