[Ffmpeg-devel] Re: [PATCH] MXF tag parser

Michael Niedermayer michaelni
Wed Aug 2 22:41:16 CEST 2006


Hi

On Wed, Aug 02, 2006 at 04:01:59PM +0200, Baptiste Coudurier wrote:
> Reimar D?ffinger wrote:
> > Hello,
> > On Wed, Aug 02, 2006 at 03:14:55PM +0200, Baptiste Coudurier wrote:
> >>> It also changes quite a few types from int to uint32_t, because
> >>> 1) I think it's more correct
> >> Isn't "int" always at least 32bit ? 

no, but a system where its not wont support ffmpeg ...


> >> If types size are exactly like in
> >> 377M I'm all for it, can you please seperate that patch from the other ?
> > 
> > Yes, but I like to it more explicit. But the "more correct" is more
> > about signed vs. unsigned.
> 
> I prefer the simplest way.

*int32_t as we all know means exactly 32bits so its not really correct

uint_fast32_t or uint_least32_t would be most correct if you want to be
pedantic, if not then int / unsigned int seems to be the logic choice ...

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is




More information about the ffmpeg-devel mailing list