Re: [PATCH] Add <sys/_stdint.h> for FreeBSD compatibility

[Sebastian Huber <sebastian dot huber at embedded-brains dot de>]

On 30/03/15 12:41, Corinna Vinschen wrote:
> On Mar 24 11:00, Corinna Vinschen wrote:
> > >On Mar 23 20:36, Sebastian Huber wrote:
> > > > >	* libc/include/sys/_stdint.h: New file.
> > > > >	* libc/include/stdint.h (int8_t): Move to <sys/_stdint.h>.
> > > > >	(uint8_t): Likewise.
> > > > >	(int16_t): Likewise.
> > > > >	(uint16_t): Likewise.
> > > > >	(int32_t): Likewise.
> > > > >	(uint32_t): Likewise.
> > > > >	(int64_t): Likewise.
> > > > >	(uint64_t): Likewise.
> > > > >	* libc/include/sys/types.h: Include <sys/_stdint.h>.
> > > > >	* libc/sys/rtems/machine/_types.h: Remove <stdint.h> include.
> > > > >	* libc/sys/time.h>: Replace __uint32_t with uint32_t and
> > > > >	__uint64_t with uint64_t.
> > >
> > >I'd like to take the opportunity to drop the Cygwin-only stdint.h
> > >and inttypes.h files and use the newlib ones instead.
> > >
> > >Does anybody have some testcode which allows to see if the old and
> > >the new stdint.h/inttypes.h render the exact same result?
> Obviously not.  I try to come up with a test this week so I can check
> that removing the Cygwin stdint.h and inttypes.h doesn't change the
> result.  Sorry, but I really don't want to apply this patch without
> the accompanying Cygwin change.  Hope that's ok.

Its good that you didn't check it in yet, since the patch was 
incomplete. I forgot to move the intptr_t and uintptr_t definitions. I 
sent a second version of the patch.

--
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.huber@embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschÃftliche Mitteilung im Sinne des EHUG.