[FFmpeg-devel] [PATCH] RoQ video encoder, take 2

[Reimar Döffinger]

Hello,
On Sun, May 27, 2007 at 11:10:30AM -0400, Dave Dodge wrote:
> On Sun, May 27, 2007 at 02:20:30PM +0200, matthieu castet wrote:
> > > Unless there is a special reason not to, try using just "int" instead of
> > > "int8_t".
> > > 
> > Yes, and for the question why, there are some architecture (like arm) 
> > where everything is 32 bits. Using int8_t force the compiler to add 
> > extra instructions to emulate 8 bits operations (overflow, ...).
> 
> If you want to indicate that this is an 8-bit value, another option
> would be to use int_fast8_t.  That should select the "fastest" integer
> that can hold at least 8 bits.

I disagree, the int_fast*_t types are useless, they rarely are related
to reality. E.g. int_fast8_t on x86 is 8 bit, but that will be significantly
slower when used as e.g. array index. Even if not it won't really be any
faster than just int.

Greetings,
Reimar D?ffinger