[PATCH 2/2] fix compile errors for efgcvt.c

Corinna Vinschen vinschen@redhat.com
Wed Jul 24 09:59:00 GMT 2019


On Jul 24 17:43, Kota Uchida wrote:
> On Wed, Jul 24, 2019 at 4:25 PM Corinna Vinschen <vinschen@redhat.com> wrote:
> > Why does your patch arbitrarily move the gcvt function?  There's
> > no functional change, so this move should not be part of the patch.
> 
> The movement is the core of this patch.
> 
> When efgcvt.c is compiled under some specific conditions,
> gcvt's function prototype is NOT provided by stdlib.h.
> (Its prototype is gurded by macros)
> 
> If gcvt is below gcvtf, gcvt is implicitly declared as int
> since gcvtf calls gcvt.
> The implicit declaration is not compatible with its actual type.
> 
> So the possible options will be:
> - To move gcvt above gcvtf, or
> - To declare gcvt's prototype above gcvtf.
> 
> Should I choose 2nd option?

No, I pushed the patch.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/newlib/attachments/20190724/47d57a76/attachment.sig>


More information about the Newlib mailing list