RFR 8170364: FilePermission path modified during merge

Alan Bateman Alan.Bateman at oracle.com
Mon Nov 28 14:17:05 UTC 2016



On 28/11/2016 08:40, Wang Weijun wrote:
> Hi Alan
>
> Updated webrev at
>
>     http://cr.openjdk.java.net/~weijun/8170364/webrev.01
>
> Changes since webrev.00:
>
> - a private constructor that can clones 4 fields and modifies 5 others
>
> - using lambda
>
> - test enhancement
>
This looks much better. A minor nit but the for readability purposes 
then it would be easy to read if the parameters to the constructor were 
all aligned rather than groups of two.

-Alan


More information about the core-libs-dev mailing list