[FFmpeg-devel] [PATCH] ALS decoder
Mon Aug 31 01:35:16 CEST 2009
Thilo Borgmann <thilo.borgmann at googlemail.com> writes:
>>>>> +/** Reads and decodes a Rice codeword.
>>>>> + */
>>>>> +static int32_t decode_rice(GetBitContext *gb, unsigned int k)
>>>>> + int max = gb->size_in_bits - get_bits_count(gb) - k;
>>>>> + int32_t q = get_unary(gb, 0, max);
>>>> i suspect int is fine for these 2 instead of int32_t ?
>>> Is int at least 32 bits long? k <= 32 and therefore q might have to hold
>>> up to 32 bits.
>> int in POSIX is at least 32bits long, yes
> This makes (u)int32_t useless, doesn't it?
- int32_t is *exactly* 32 bits (and it's optional)
- int could be much wider than 32 bits. In this case, if another
type (short, char, or a compiler-specific type) is 32 bits, this will
be used for int32_t.
mans at mansr.com
More information about the ffmpeg-devel