[FFmpeg-devel] [PATCH] merge audio and video bitstream filters option parsing
Guillaume LECERF
foxcore
Fri Oct 12 14:51:35 CEST 2007
Hi, just a small question :
2007/10/12, Benoit Fouet <benoit.fouet at purplelabs.com>:
> +static void opt_bsf(const char *opt, const char *arg)
...
> - bsfp= &video_bitstream_filters;
> + bsfp= *opt == 'v' ? &video_bitstream_filters : &audio_bitstream_filters;
...
> - { "absf", HAS_ARG | OPT_AUDIO | OPT_EXPERT, {(void*)opt_audio_bsf}, "", "bitstream filter" },
> - { "vbsf", HAS_ARG | OPT_VIDEO | OPT_EXPERT, {(void*)opt_video_bsf}, "", "bitstream filter" },
> + { "absf", OPT_FUNC2 | HAS_ARG | OPT_AUDIO | OPT_EXPERT, {(void*)opt_bsf}, "", "bitstream filter" },
> + { "vbsf", OPT_FUNC2 | HAS_ARG | OPT_VIDEO | OPT_EXPERT, {(void*)opt_bsf}, "", "bitstream filter" },
Doesn't it need "v" as the second default parameter of the opt_bsf()
function for the vbsf parameter ?
--
Guillaume LECERF
GeeXboX developer - www.geexbox.org
More information about the ffmpeg-devel
mailing list