[PATCH] Harmonize stdint.h and _default_types.h

Hans-Peter Nilsson hans-peter.nilsson@axis.com
Tue Dec 3 14:48:00 GMT 2013


> From: Corinna Vinschen <vinschen@redhat.com>
> Date: Tue, 3 Dec 2013 15:11:15 +0100

> this patch looks good to me.  Hans-Peter, would you mind to have a look
> as well?

Thanks.  These look odd:

> +++ b/newlib/libc/include/stdint.h

> +#ifdef INT8_C
> +#define INT8_C(x) __INT8_C(x)
> +#define UINT8_C(x) __UINT8_C(x)
> +#else
>  #define INT8_C(x)	x
>  #if __STDINT_EXP(INT_MAX) > 0x7f
>  #define UINT8_C(x)	x
>  #else
>  #define UINT8_C(x)	x##U
>  #endif
> +#endif

It doesn't look like you want to redefine INT8_C (in which case
better #undef it) but rather like you meant "#ifdef __INT8_C"
for the first line.

Same for the other *_C variants, except for INTMAX_C
(__INTMAX_C) where it is as expected.

Otherwise, nothing suspicious besides the include stdint.h in
libc/sys/rtems/machine/_types.h, but which the comment says is a
conscious choice.

brgds, H-P



More information about the Newlib mailing list