RFR 8153123 : Streamline StackWalker code

Mandy Chung mandy.chung at oracle.com
Wed Apr 6 05:36:23 UTC 2016


> On Apr 5, 2016, at 4:48 PM, Brent Christian <brent.christian at 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


More information about the core-libs-dev mailing list