[FFmpeg-devel] [PATCH] lavfi: reimplement MPlayer's af_pan filter for libavfilter.

Nicolas George nicolas.george at normalesup.org
Thu Nov 17 11:51:19 CET 2011


Le septidi 27 brumaire, an CCXX, Stefano Sabatini a écrit :
> At your discretion you may want to change authorship at this point,
> and mention Clement in the commit message.

I tweaked the commit message.

> Nit++++: (C) -> (c) for consistency

Done.

> this comment may be more explicit, or entirely removed

Done.

> please rename channel to something like channel_nb or add a
> comment (channel as is suggest a channel layout with a single channel)

channel_id, since it is called that elsewhere.

> I'd suggest nb_channels here, "number of named" is quite confusing,

nb_in_channels.

> maybe together with #define NAMED 1.

This seems overkill.

> Note: I wonder if we have a function which counts bits

av_get_channel_layout_nb_channels does it, in fact. But there are arithmetic
tricks that work well to speed up things, it may be a good addition.

> maybe add a comment about what degenerate means in this context

Done.

> Nit: printing some context information (e.g. gain value and indices i
> j) may be helpful here

Good remark, done.

> av_clipf?

I looked for it but did not find it at first, thanks. I also fixed the
interval, the limit was 128 and not 256 (although it would probably fit up
to 255, in fact).

New version incoming.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111117/3eef1e1a/attachment.asc>


More information about the ffmpeg-devel mailing list