> 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