Re: RFR 8170364: FilePermission path modified during merge

2016-11-28 Thread Alan Bateman
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

Re: RFR 8170364: FilePermission path modified during merge

2016-11-28 Thread Wang Weijun
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 Thanks Max

Re: RFR 8170364: FilePermission path modified during merge

2016-11-27 Thread Wang Weijun
> On Nov 27, 2016, at 7:13 PM, Wang Weijun wrote: > >> >> On Nov 27, 2016, at 6:12 PM, Alan Bateman wrote: >> >> On 26/11/2016 08:54, Wang Weijun wrote: >> >>> Please take a review at >>> >>>

Re: RFR 8170364: FilePermission path modified during merge

2016-11-27 Thread Alan Bateman
On 26/11/2016 08:54, Wang Weijun wrote: Please take a review at http://cr.openjdk.java.net/~weijun/8170364/webrev.00/ The compatibility layer introduced in the new FilePermission implementation requires one FilePermission to imply another with either a relative path or an absolute path.