[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