[FFmpeg-devel] [PATCH 2/3] lavfi/buffersink: accept parameters as options.

[Nicolas George]

Le duodi 22 germinal, an CCXXI, Stefano Sabatini a écrit :
> Nit: you could merge check and log error in the same macro, maybe save
> some lines.

Indeed: patch sent (not attached to this thread, but it is rather trivial).

> Also again, if the list is always set by av_opt_set_int_list(), how
> could size be wrong?

There is nothing preventing users from using av_opt_set_bin() directly,
especially if they have the lists in a different format than the fields in
the AVCodec structures.

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/20130413/77e0f057/attachment.asc>