[FFmpeg-devel] [PATCH 1/2] lavfi: add astreamsync audio filter.
Nicolas George
nicolas.george at normalesup.org
Sat Dec 24 18:02:20 CET 2011
Le quartidi 4 nivôse, an CCXX, Stefano Sabatini a écrit :
> not needed?
Indeed. Neither were the next two ones.
> please provide some comment, or it will harder to read/understand this
> code
Done.
> maybe note that this is a bitmap, one bit for each input
> pad/link (maybe using an array instead may help readability)
Done.
> > + for (i = 0; i < 42; i++)
> > + av_expr_eval(as->expr, as->var_values, NULL); /* exercize prng */
> uh?
The first value returned by random(1) seems to be always 0. That does not
look really random.
> nit: qid for making it more clear that id refers a queue (pad_id ->
> pid or link_id -> lid would be fine as well)
Done.
> nit:
> *queue = as->queue[id];
> should ease readability:
Good idea, done.
> circular buffer overflow? shouldn't you check that the buffer is not
> already filled (and in that case maybe release memory)?
That is not necessary: just after this code, which is the only place where
the queue grows, there is a call to send_next, and send_next always ensure
that the buffer is not full.
> Nit: missing final dot.
Done.
Also, config_output was not doing its job, neither was it in amerge: that is
fixed.
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/20111224/00d3f13f/attachment.asc>
More information about the ffmpeg-devel
mailing list