<br><br><div class="gmail_quote">On Tue, Oct 27, 2009 at 7:32 AM, Maciej Pijanka <span dir="ltr"><<a href="mailto:maciej.pijanka@gmail.com">maciej.pijanka@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div></div><div class="h5">On 27/10/2009, Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> wrote:<br>
><br>
> Maciej,<br>
><br>
> Thanks for the patch.  I think most of it is ready to be committed.<br>
><br>
> Index: src/lib/clog2.c<br>
> ===================================================================<br>
> --- src/lib/clog2.c   (revision 4869)<br>
> +++ src/lib/clog2.c   (working copy)<br>
> @@ -7,6 +7,8 @@<br>
>  /* Assume 8 bits per byte */<br>
>  #define CHAR_BIT 8<br>
><br>
> +unsigned long log2(unsigned long x);<br>
> +<br>
>  unsigned long log2(unsigned long x)<br>
>  {<br>
>          // assume 8 bits per byte.<br>
><br>
> Things like this make me wonder if we should just turn off the warning.  Is<br>
> there a header file where we can put some of these prototypes?<br>
<br>
</div></div>On begining of previous patch version, was an comment about such<br>
things, i didn't found<br>
suitable header where such utility function prototypes might be added,<br>
if its desired to make<br>
include/utils.h or something similar with better name i will move<br>
prototypes there.<br></blockquote><div>Sorry, I didn't look at the first version of the patch.  I think that the cleanest thing would be to have files in include that mirror the .c files.  If we have to add too many new .h files, maybe we should combine some .c files.<br>
<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
But please note in few places prototype is just before function<br>
because it is used only locally (for my knowledge) if not then it<br>
should be moved to header too.<br></blockquote><div>If you declare it as static that will silence the warning as well.  Of course this only works when it is just used locally.<br><br>Thanks,<br>Myles<br></div></div><br>