[FFmpeg-cvslog] r11211 - trunk/libavcodec/i386/dsputil_mmx.h

Diego Biurrun diego
Tue Dec 18 23:40:49 CET 2007


On Tue, Dec 18, 2007 at 09:50:02PM +0100, Aurelien Jacobs wrote:
> Diego Biurrun wrote:
> 
> > On Wed, Dec 12, 2007 at 11:45:03PM +0100, aurel wrote:
> > > 
> > > Log:
> > > add required include to make this file self-contained
> > > 
> > > --- trunk/libavcodec/i386/dsputil_mmx.h	(original)
> > > +++ trunk/libavcodec/i386/dsputil_mmx.h	Wed Dec 12 23:45:03
> > > 2007 @@ -22,6 +22,8 @@
> > >  #ifndef FFMPEG_DSPUTIL_MMX_H
> > >  #define FFMPEG_DSPUTIL_MMX_H
> > >  
> > > +#include <stdint.h>
> > > +
> > >  extern const uint64_t ff_bone;
> > >  extern const uint64_t ff_wtwo;
> > 
> > Isn't inttypes.h more portable and thus preferable?
> 
> $ find ffmpeg -name '*.h' -exec grep -H stdint.h {} \; | wc -l
> 70

I am aware that we already use stdint.h in a lot of places.  Still it
could make sense to switch to inttypes.h where possible if it is more
portable.

Diego




More information about the ffmpeg-cvslog mailing list