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

Nicolas George nicolas.george at normalesup.org
Sun Nov 6 17:28:11 CET 2011


Le sextidi 16 brumaire, an CCXX, Clément Bœsch a écrit :
> Since you changed the variable into 'l', you should update the 'n'
> reference here.

Indeed. I reformulated the sentence.

> You should add yourself here now :)

Ok.

> I'm not familiar enough with fixed point arithmetic; what does the '+'
> stand for?

It is supposed to mean that this is signed arithmetic. But this was a
notation I invented on the spot. I looked up in wikipedia, and the only
notation they give for signedness is "1:7:8", which I adopted.

> Shouldn't this be /256?

Of course. Sorry.

> Does it still work as expected when the number of specified input channels
> doesn't match the actual number of input channels?

It would not have, that was a point that needed discussion. On the
afterthought, I think that the actual number of input channels is more
important than the declared one, and I adapted the code.

It was a small discomfort that filters have no method called after
query_formats, when the filtergraph has decided the various formats.

> Feel free to remove this chunk as well if it works.

Indeed.

> Pointless braces?

I avoid omitting outer braces when the inner block is complex, because I
find it more risky and less readable.

> Anyway, thanks a lot for submitting quickly this patchset.

Thanks to you for the original work.

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/20111106/ab236bac/attachment.asc>


More information about the ffmpeg-devel mailing list