[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