> > Added the following defines to compiler.h These are c99 constructs > > that we don't have in our gcc headers. > > > > #define PRIu32 "u" > > #define PRIX32 "X" > > #define PRId32 "d" > > #define PRIX64 "X" > > #define PRId64 "d" > > #define PRIu64 "u" > > #define PRIx64 "x" > > #define PRIx32 "x" > > #define PRIx16 "x" > > #define PRIx8 "x" > > > > My understanding is that the 64bit values need to have the length > specified > to output proper values. For 32-bit this would be: > > #define PRIX64 "llX" > #define PRId64 "lld" > #define PRIu64 "llu" > #define PRIx64 "llx" > > > #define INT32_MAX 2147483647 > > #define UINT64_C(val) val##ULL > > What it should be for 32-bit. Code style wise I prefer using value > instead > of val, but that's nitpicking on my part. ;) We should really add those, ffmpeg insists on them as well, and I've a hard time trying to push default values to svn (some freaks want people to port their OSes to ffmpeg instead of the other way round). It's from the standard anyway so it shouldn't hurt. François.