[FFmpeg-devel] [PATCH] NellyMoser audio decoder v2

Benjamin Larsson banan
Sat Oct 13 19:17:36 CEST 2007


Vitor Sessak wrote:
> Hi
> 
> Just some nitpicking...
> 
> Benjamin Larsson wrote:
>> Michael Niedermayer wrote:
>>
>> static inline int signed_shift(int i, int shift) {
>>     if (shift > 0) {
>>         return i << shift;
>>     }
> 
> Indentation in the patch is inconsistent between
> 
> if (a) {
>      b;
> }
> 
> and
> 
> if (a)
>      b;
> 
> I personally prefer the latter.
> 
>> AVCodec nellymoser_decoder = {
>>     "nellymoser",
>>     CODEC_TYPE_AUDIO,
>>     CODEC_ID_NELLYMOSER,
>>     sizeof(NellyMoserDecodeContext),
>>     decode_init,
>>     NULL,
>>     decode_end,
>>     decode_tag,
>>     0,
>> };
> 
> This 0 is useless.
> 
> -Vitor

Thanks fixed in local tree.

MvH
Benjamin Larsson




More information about the ffmpeg-devel mailing list