<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><span style="font-family:arial,sans-serif">On Fri, Apr 8, 2016 at 3:03 AM, Dave Chinner </span><span dir="ltr" style="font-family:arial,sans-serif"><<a href="mailto:david@fromorbit.com" target="_blank">david@fromorbit.com</a>></span><span style="font-family:arial,sans-serif"> wrote:</span><br></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Apr 07, 2016 at 07:41:58PM -0500, Eric Sandeen wrote:<br>
> On 4/7/16 5:50 AM, Jan Tulak wrote:<br>
> > On Wed, Apr 6, 2016 at 11:10 PM, Eric Sandeen <<a href="mailto:sandeen@sandeen.net">sandeen@sandeen.net</a> <mailto:<a href="mailto:sandeen@sandeen.net">sandeen@sandeen.net</a>>>wrote:<br>
</span><span class="">> >     (hm, why do we have 3 copies of cvtnum?)<br>
> ><br>
> ><br>
> > ​Where? ​<br>
> >  ​<br>
> ><br>
> > ​$ pwd<br>
> > ​/some/path/xfsprogs-dev<br>
> ><br>
> > $ grep -r "cvtnum(size_t" . |grep -v "^Binary"<br>
> > ​./include/input.h:extern long long cvtnum(size_t blocksize, size_t sectorsize, char *s);<br>
> > ./io/init.h:extern void init_cvtnum(size_t *blocksize, size_t *sectsize);<br>
><br>
>   File                Line<br>
> 0 estimate/xfs_estimate.c   29 cvtnum(char *s)<br>
> 1 libxcmd/input.c          147 cvtnum(<br>
> 2 mkfs/xfs_mkfs.c         3618 cvtnum(<br>
<br>
</span>FWIW, here's why we put the function name at the start of the line,<br>
rather than the way that it is typically done in the kernel code<br>
(i.e. after the return type specification):<br>
<br>
$ git grep ^cvtnum<br>
estimate/xfs_estimate.c:cvtnum(char *s)<br>
libxcmd/input.c:cvtnum(<br>
mkfs/xfs_mkfs.c:cvtnum(<br>
<br></blockquote><div><br></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​Ah, I see. A nice trick. :-)</div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline"><br></div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">It might be good to merge the three functions, but I think it can wait for when this patchset is done - or there is always going to be something more to add... </div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline"><br></div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">Thanks,</div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">Jan ​</div> </div></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Jan Tulak<br></div><a href="mailto:jtulak@redhat.com" target="_blank">jtulak@redhat.com</a> / <a href="mailto:jan@tulak.me" target="_blank">jan@tulak.me</a></div></div></div></div>
</div></div>