This is the mail archive of the
newlib@sourceware.org
mailing list for the newlib project.
Re: Sebastian's Patches Committed
- From: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
- To: joel dot sherrill at oarcorp dot com
- Cc: newlib at sourceware dot org, Sebastian dot Huber at embedded-brains dot de, chrisj at rtems dot org
- Date: Fri, 25 Oct 2013 04:30:26 +0200
- Subject: Re: Sebastian's Patches Committed
- Authentication-results: sourceware.org; auth=none
> From: Joel Sherrill <joel.sherrill@oarcorp.com>
> Date: Tue, 15 Oct 2013 19:41:34 +0200
> I just committed Sebastian's patches. If there are issues,
> just speak up. :)
This seems wrong. Including e.g. stdlib.h shouldn't pull in
stdint.h. There's likely standards language forbidding this but
I don't have it right here. Glibc used to have such issues.
Such bleed is not helpful to users; they should need to include
the right headers, not just a random header when needing
e.g. uint32_t.
Right, it affects only margin cases, but those exist. See for
example gcc/testsuite/gcc.dg/20050922-1.c and -2.c on the 4.7
and 4.8 branch (they might be "fixed" soon to not define
uint32_t).
brgds, H-P