RFR 8168127: FilePermissionCollection merges incorrectly

Roger Riggs Roger.Riggs at Oracle.com
Tue Oct 25 14:12:02 UTC 2016


Hi Max,

It might be worth a comment (Lines 258 and 270) that the names are 
generated to be different than the original.
Otherwise, looks fine.

Roger



On 10/19/2016 10:47 PM, Wang Weijun wrote:
> Please review the code change at
>
>     http://cr.openjdk.java.net/~weijun/8168127/webrev.00/
>
> Two changes:
>
> 1. npath2 is considered in equals and hashCode of FilePermission, so 2 objects with different npath2 can be added to a map and different entries.
>
> 2. special name for newPermUsingAltPath and newPermPlusAltPath results, so FilePermissionCollection::add will not merge one with the original.
>
> Thanks
> Max
>



More information about the core-libs-dev mailing list