> On Mar 15, 2017, at 1:30 PM, Alan Bateman <alan.bate...@oracle.com> wrote:
> 
> On 15/03/2017 19:42, Mandy Chung wrote:
> 
>> StackWalker::getInstance is currently specified to check for 
>> StackFramePermission("retainClassReference“) due to an early review feedback 
>> .  Given it has only one target, it’s overkill to define a specific 
>> permission type for stack walking use.   This patch proposes to replace the 
>> StackWalker permission check with 
>> RuntimePermission("getStackWalkerWithClassReference”) and remove 
>> StackFramePermission class.
>> 
>> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8176815/webrev.00/
>> 
> The target name is long but I agree with removing this permission (and the 
> changes look good).

An alternative target name is “getStackWalker” that I suggested previously but 
doesn’t seem attracting to other.  It’s hard to anticipate if a different 
target name is needed in the future.

Another reason why I’m okay with this long name because it’s one character less 
than another target name:

  *   <td>setDefaultUncaughtExceptionHandler</td>

Mandy

Reply via email to