[Nestmates] RFR: 8199309: [Nestmates] The new Class nestmate methods should have SecurityManager checks

David Holmes david.holmes at oracle.com
Tue Mar 13 05:48:53 UTC 2018


On 13/03/2018 3:41 PM, John Rose wrote:
> On Mar 12, 2018, at 7:44 PM, David Holmes <david.holmes at oracle.com> wrote:
>>
>> So I don't see how this "sanity check" helps anything. ?? Plus it leads to confusion for anyone later reading the code.
> 
> That’s funny. I was confused wondering how to prove,locally, that the value of members[0] was ‘this’. IDifferent folks find different things confusing.

members[0] doesn't have to be "this". members[0] is always the nesthost 
and that will only be "this" if getNestMembers() is called on the 
nesthost class.

> Suggest adding an assert to getNestMembers that the host always comes first.

I can do that.

> That would serve as local proof that the code which confused me is valid.

Thanks,
David




More information about the valhalla-dev mailing list