> On Apr 20, 2017, at 5:12 PM, Andrey Nazarov <andrey.x.naza...@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

Reply via email to