[FFmpeg-devel] [PATCH] ffmpeg_opt: Add -hwaccels option that lists all supported hwaccels
gajjanag at mit.edu
Wed Aug 26 03:25:14 CEST 2015
On Tue, Aug 25, 2015 at 9:03 PM, Timothy Gu <timothygu99 at gmail.com> wrote:
> On Tue, Aug 25, 2015 at 08:19:59PM -0400, Ganesh Ajjanagadde wrote:
>> For consistency, prefix with an opt_?
> This name is chosen to be consistent with other show_ functions. opt_*
> are exclusively used for setting some options AFAICT.
Ok. Please place the function outside of all the opt_* stuff;
since this is not setting options, it should not be in the middle of them.
Otherwise, patch LGTM.
>> Maybe you could check the return value of printf?
> Absolutely zero calls to printf in FFmpeg are checked. Even if checking
> the result code can be more secure, it is way outside the scope of this
Perhaps. My point was not so much from the position of security,
but program correctness: a non zero printf return should propagate to
the exit code.
You are right this is beyond this patch's scope.
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
More information about the ffmpeg-devel