RFR 8178323: Add negative tests for bind services Jlink feature
    Mandy Chung 
    mandy.chung at oracle.com
       
    Fri Apr 21 00:46:01 UTC 2017
    
    
  
> On Apr 20, 2017, at 5:12 PM, Andrey Nazarov <andrey.x.nazarov at oracle.com> wrote:
> 
> HI Mandy,
> 
> I’ve update patch http://cr.openjdk.java.net/~anazarov/8178323/webrev.03/webrev/ <http://cr.openjdk.java.net/~anazarov/8178323/webrev.03/webrev/>
Nit: use 4-space indent e.g. BindServices.java. line 159 (the following lines will move along) and SuggestProviders.java line 255-258, 262, 273-276,  and 280.
Otherwise looks good.
You can fix the formatting before push.
Thanks
Mandy
    
    
More information about the jigsaw-dev
mailing list