[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