[FFmpeg-devel] [PATCH] cmdutils: include type in filter list.
nicolas.george at normalesup.org
Thu Jan 19 21:27:15 CET 2012
Le decadi 30 nivôse, an CCXX, Stefano Sabatini a écrit :
> nit: get_media_type_char() (consistent with av_get_picture_type_char() for example)
> Please split the get_media_type_letter() change from the opt_filters()
> suggestion: at your discretion, desc and descp for showing the
> correlation between the two variables
> This check can be moved upward, also it is too restrictive on the
> second run but this is just a nit (unrelevante from the practical POV,
> so do as you prefer).
At first glance, I fear it will be more complicated, for no practical gain.
> Looks fine otherwise.
Thanks. Here are the new patches. The next four days will be very busy for
me, so feel free to apply if you want.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel