On 05/25/2016 12:49 PM, Alan Bateman wrote:


On 25/05/2016 20:01, Xueming Shen wrote:
:

Sure, if there is such "plan".

http://cr.openjdk.java.net/~sherman/8139956_8146754_8147539_8153248/webrev

Here is the updated webrev with the @module tag.

In ZipPath.register then it would be good to include a comment to say something 
"watcher must be associated with a different provider" as it might not be 
immediately clear why ProviderMismatchException is thrown.

The rest looks good to me.

-Alan

thanks! updated accordingly.

http://cr.openjdk.java.net/~sherman/8139956_8146754_8147539_8153248/webrev

-sherman

Reply via email to