[MPlayer-dev-eng] [PATCH] rtsp.c and rmff.h

[Diego Biurrun]

Roberto Togni wrote:
> On 2003.04.18 23:14 Steven M. Schultz wrote:
>>     Does Cygwin have inttypes.h?  If so then including
>> <inttypes.h>
>>     would be better than '#ifdef __CYGWIN__".
>>
> Can someone with Cygwin comments on this? inntypes.h seems to be 
> included in other MPlayer files with no problem. Are those ifdef needed?
> Code compiles fine here (Hacked Debian woody, gcc 2.95.4)

It compiles fine here on Cygwin after I remove the following lines from 
all files in the realrtsp directory:

#ifdef __CYGWIN__
#define uint32_t unsigned int
#define uint16_t unsigned short int
#define uint8_t unsigned char
#endif

And to me this looks like a duplication of etc/cygwin_inttypes.h (which 
has to be put in /usr/include/inttypes.h on Cygwin to make MPlayer 
compile).  Here it is:

#if 0
typedef char *  caddr_t;
typedef char int8_t;
typedef unsigned char u_int8_t;
typedef short int16_t;
typedef unsigned short u_int16_t;
typedef int int32_t;
typedef unsigned int u_int32_t;
typedef long long int64_t;
typedef unsigned long long u_int64_t;
typedef int32_t register_t;
#else
#include <sys/types.h>
#endif /* 0/1 */
typedef u_int8_t uint8_t;
typedef u_int16_t uint16_t;
typedef u_int32_t uint32_t;
typedef u_int64_t uint64_t;

So I guess it should be safe to remove those lines.

Diego