RFR: jmod remove
Alan Bateman
Alan.Bateman at oracle.com
Thu May 10 08:36:25 PDT 2012
On 09/05/2012 15:31, Chris Hegarty wrote:
> Alan,
>
> I finally got back to this. Comments inline...
>
> Updated webrev:
> http://cr.openjdk.java.net/~chegar/jigsaw/jmodrm.05/webrev/
We've gone several rounds on this and I think I'm mostly mostly happy.
Mandy - do you plan to do another review? I'm happy to push this for
Chris once it is final.
A couple of small points:
Files.deleteTreeUnchecked - the code is still catching the "useful"
IOException and throwing a less useful IOException. I would suggest just
replacing each one with execs.add(x).
In SimpleLibrary.moduleTrashDir it uses SecureRandom but I think that is
overkill. A sequence number of java.util.Random should be fine.
-Alan.
More information about the jigsaw-dev
mailing list