[newlib] print formats for FAST and LEAST types
Andre Vieira
Andre.SimoesDiasVieira@arm.com
Fri Jul 24 10:49:00 GMT 2015
On 23/07/15 20:43, Corinna Vinschen wrote:
> Hi Andre,
>
> On Jul 23 11:28, Andre Vieira wrote:
>> The PRI and SCN macro's were producing formats that did not match their
>> target types set by GCC. This patch uses the types defined for
>> __INTxx_TYPE__, __INT_FASTxx_TYPE__ and __INT_LEASTxx_TYPE__ to define their
>> corresponding macros.
>>
>> newlib/ChangeLog:
>> 2015-07-23 Andre Vieira <...>
>>
>> * libc/include/sys/_intsup.h: Defined new __INTxx, __FASTxx and
>> _LEASTxx macro's to hold information regarding the respective types
>> print and scan formats.
>> * libc/include/inttypes.h: Defined LEAST and FAST specific PRI and SCN
>> macro's as these are not always the same as the INT variants. Used
>> the new
>> __INTxx, __FASTxx and __LEASTxx macro's in their corresponding PRI
>> and SCN macros.
>
> I gave your patch a quick glance and it looks basically ok to me.
> I'll review it more thorougly tomorrow (I hope), but I have a question:
>
> How did you test your patch?
>
>
> Thanks,
> Corinna
>
Hi Corinna,
That is a very good question. To be honest with you I ran gcc and newlib
regression tests and did a manual test on one machine. As I was about to
send you the manual patch I noticed I forgot to change the SCN16(x)
define. So I'll respin this patch for you and I'll also give testing a
bit more thinking.
There is no straightforward way of testing this I think. Though I will
try to create a sensible sscanf/sprintf test and get back to you.
Suggestions are welcome.
Thanks,
Andre
More information about the Newlib
mailing list