[MPlayer-cvslog] r19407 - in trunk: libass/ass_mp.c libass/ass_mp.h libass/ass_render.c mplayer.c

Uoti Urpala uoti.urpala at pp1.inet.fi
Tue Aug 15 23:25:43 CEST 2006


On Wed, 2006-08-16 at 01:13 +0400, Evgeniy Stepanov wrote:
> On Wednesday 16 August 2006 00:47, Reimar Döffinger wrote:
> > Sorry for missing it in previous reviews, but AFAIK in MPlayer you're
> > supposed to include inttypes.h, not stdint.h.
> 
> It is used much more often, at the least :)
> Is there any reason for this ? They are both standard headers, inttypes.h 
> includes stdint.h (at least in my system), so why not use stdint.h when it is 
> sufficient ?

Yes according to the standard stdint.h is a subset of inttypes.h; stdint
contains the basic types, inttypes some extra and the PRI macros. So it
only matters on some old nonstandard systems (I don't know what those
might be).

Including both like ass_render.c and ass.c do is certainly redundant :)




More information about the MPlayer-cvslog mailing list