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