Simple local.h patch

Jeff Johnston jjohnstn@redhat.com
Tue Jun 16 18:06:00 GMT 2009


Howland Craig D (Craig) wrote:
> Attached is a simple patch to avoid prototype warnings.
>  
> Note that the same prototype is in 2 different files.  I did not correct
> that in this patch, but a good question is how to.  __locale_charset()
> is defined in libc/locale/locale.c.  Both of these prototypes are in
> different directories.  It would be the most work, but it seems like
> the best approach would be to create a local.h in locale that could
> be included in the ctype and stdlib local.h files.  Thoughts?
>
> Craig
>  
> 	* libc/ctype/local.h (__locale_charset):  add arguments to
> prototype
> 	* libc/stdlib/local.h (__locale_charset):  ditto
>   
Patch checked in.

In general, the local.h headers are not meant to cross directories.  
They are allowed to define things without worrying if another directory 
has chosen to define the same symbols in their local.h.

I have no problem with the current situation, but we could conceivably 
create another header in libc/locale (e.g. internal_locale.h) and have 
the local.h files in question include that header.

-- Jeff J.



More information about the Newlib mailing list