[FFmpeg-devel] [PATCH] NellyMoser audio decoder v2
Vitor Sessak
vitor1001
Sat Oct 13 16:31:16 CEST 2007
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
More information about the ffmpeg-devel
mailing list