Re: [9] Review request: 8154203: Use StackWalker instead of the now-deprecated sun.reflect.Reflection class

2016-04-14 Thread Mandy Chung 

> On Apr 14, 2016, at 3:27 PM, Kevin Rushforth  
> wrote:
> 
> Hi Vadim,
> 
> Please review:
> 
> https://bugs.openjdk.java.net/browse/JDK-8154203
> http://cr.openjdk.java.net/~kcr/8154203/webrev.00/
> 
> It's a straight-forward fix to replace Reflection::getCallerClass with 
> StackWalker::getCallerClass and remove the (no longer needed) CallerSensitive 
> annotation.

+1

It’s good to see this migrate to use StackWalker.

Mandy

[9] Review request: 8154203: Use StackWalker instead of the now-deprecated sun.reflect.Reflection class

2016-04-14 Thread Kevin Rushforth 

Hi Vadim,

Please review:

https://bugs.openjdk.java.net/browse/JDK-8154203
http://cr.openjdk.java.net/~kcr/8154203/webrev.00/

It's a straight-forward fix to replace Reflection::getCallerClass with 
StackWalker::getCallerClass and remove the (no longer needed) 
CallerSensitive annotation.


-- Kevin