Various cleanups, corrected patch

[Howland Craig D (Craig)]

There was a mistake in the stdio.h patch that causes a compile error.
A corrected patch file is attached.  Use this one in place of the
previous email's patch file.
				Craig


-----Original Message-----
From: newlib-owner@sourceware.org [mailto:newlib-owner@sourceware.org]
On Behalf Of Howland Craig D (Craig)
Sent: Friday, November 21, 2008 5:11 PM
To: newlib@sourceware.org
Subject: Various cleanups

     I just did a little bit of cleanup, mostly getting rid of compiler
warnings.  In doing so I found some mistakes or oversights that I also
cleaned up.  The descriptions are in the ChangeLog portion of the
attached patch file.
     There are a couple other things that I noted but did not fix,
pending getting someone to help answer some questions:

1)  libc/stdio/mktemp.c calls _stat64_r(), but _stat64_r() does not
appear anywhere except in mktemp.c; it appears to be missing.  It seems
that the solution is to create reent/stat64r.c (by analogy with
_fstat64_r()).
Is this the right solution, or should something else be done?

2)  libc/search/hash_func.c defines four hash functions, hash1 through
hash4.  They are declared static.  1-3 are not used.  Since they are
static, it would appear that they are therefore completely worthless as
it is impossible to access them.
Should they just be commented out?

Craig
-------------- next part --------------
A non-text attachment was scrubbed...
Name: misc.patch
Type: application/octet-stream
Size: 10630 bytes
Desc: misc.patch
URL: <http://sourceware.org/pipermail/newlib/attachments/20081124/9d08ab94/attachment.obj>