Hi Alan,

thanks for the review, I will definitely fix any formatting issue before
pushing the patch. My local repo code is properly formatted and i was
suspecting that  webrev is ignoring the space  while generating the patch
file that's why you are seeing the formatting issue.

My local code is properly formatted  and i am assuming that while pushing
there will not be any formatting issue.

Please let me know if I am missing something.

thanks,
Vyom

On Wed, Jul 15, 2020 at 12:44 AM Alan Bateman <alan.bate...@oracle.com>
wrote:

>
>
> On 14/07/2020 20:09, Daniel Fuchs wrote:
> > On 12/07/2020 07:36, Vyom Tiwari wrote:
> >> Hi Patrick,
> >>
> >> Thanks for testing, Alan, Daniel can i get the final  review comment
> >> from you both ?.
> >>
> >
> > Hi Vyom,
> >
> > http://cr.openjdk.java.net/~vtewari/8237858/webrev1.0/index.html
> Vyom - will you fix the formatting issues in linux_close.c and
> bsd_close.c before you push this? You mentioned that it's a webrev issue
> but I don't think so because they are in the patch file too.
>
> -Alan
>


-- 
Thanks,
Vyom

Reply via email to