Code Review for JEP 259: Stack-Walking API

[Mandy Chung]

> On Nov 17, 2015, at 10:42 AM, Daniel Fuchs <daniel.fuchs at oracle.com> wrote:
> 
> On 17/11/15 01:13, Mandy Chung wrote:
>> I’d like to get the code review done by this week.
>> 
>> I renamed the static factory method from create to getInstance since “create” implies to create a new instance but the method returns a cached instance instead.  I also changed the spec of getCallerClass per [1].  There is not much change since webrev.01.
>> 
>> Webrev:
>>  http://cr.openjdk.java.net/~mchung/jdk9/jep259/webrev.02
>> 
>> javadoc:
>>  http://cr.openjdk.java.net/~mchung/jdk9/jep259/api/
>> 
>> Mandy
>> [1] http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-November/036589.html
>> 
> 
> Looks good to me Mandy.
> Though I wonder whether we really need this optimization.

Thanks for the review.

Caching of StackWalker instances is not highly needed and I want to avoid the method name to preclude such possibility.

Mandy