From 20176cbc737a4896f10d199d5c02ae13cc74d34b Mon Sep 17 00:00:00 2001 From: Michael Niedermayer Date: Wed, 21 May 2008 23:41:17 +0000 Subject: [PATCH] Get rid of more senseless wrapper functions and use OPT_EXIT instead. Originally committed as revision 13233 to svn://svn.ffmpeg.org/ffmpeg/trunk --- ffmpeg.c | 8 +------- ffplay.c | 16 ++-------------- ffserver.c | 11 ++--------- 3 files changed, 5 insertions(+), 30 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index b30ecbceb..995a77545 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -3756,17 +3756,11 @@ static int opt_bsf(const char *opt, const char *arg) return 0; } -static void opt_show_version(void) -{ - show_version(); - av_exit(0); -} - static const OptionDef options[] = { /* main options */ { "L", OPT_EXIT, {(void*)show_license}, "show license" }, { "h", OPT_EXIT, {(void*)show_help}, "show help" }, - { "version", 0, {(void*)opt_show_version}, "show version" }, + { "version", OPT_EXIT, {(void*)show_version}, "show version" }, { "formats", OPT_EXIT, {(void*)opt_show_formats}, "show available formats, codecs, protocols, ..." }, { "f", HAS_ARG, {(void*)opt_format}, "force format", "fmt" }, { "i", HAS_ARG, {(void*)opt_input_file}, "input file name", "filename" }, diff --git a/ffplay.c b/ffplay.c index 02d0dc463..54b6e41c8 100644 --- a/ffplay.c +++ b/ffplay.c @@ -2432,21 +2432,9 @@ static int opt_thread_count(const char *opt, const char *arg) return 0; } -static void opt_show_help(void) -{ - show_help(); - exit(0); -} - -static void opt_show_version(void) -{ - show_version(); - exit(0); -} - static const OptionDef options[] = { - { "h", 0, {(void*)opt_show_help}, "show help" }, - { "version", 0, {(void*)opt_show_version}, "show version" }, + { "h", OPT_EXIT, {(void*)show_help}, "show help" }, + { "version", OPT_EXIT, {(void*)show_version}, "show version" }, { "x", HAS_ARG | OPT_FUNC2, {(void*)opt_width}, "force displayed width", "width" }, { "y", HAS_ARG | OPT_FUNC2, {(void*)opt_height}, "force displayed height", "height" }, { "s", HAS_ARG | OPT_VIDEO, {(void*)opt_frame_size}, "set frame size (WxH or abbreviation)", "size" }, diff --git a/ffserver.c b/ffserver.c index 08a73be96..bc45a0e0e 100644 --- a/ffserver.c +++ b/ffserver.c @@ -4333,12 +4333,6 @@ static void handle_child_exit(int sig) need_to_start_children = 1; } -static void opt_show_license(void) -{ - show_license(); - exit(0); -} - static void opt_debug() { ffserver_debug = 1; @@ -4351,12 +4345,11 @@ static void opt_show_help(void) "Hyper fast multi format Audio/Video streaming server\n"); printf("\n"); show_help_options(options, "Main options:\n", 0, 0); - exit(0); } static const OptionDef options[] = { - { "h", 0, {(void*)opt_show_help}, "show help" }, - { "L", 0, {(void*)opt_show_license}, "show license" }, + { "h", OPT_EXIT, {(void*)opt_show_help}, "show help" }, + { "L", OPT_EXIT, {(void*)show_license}, "show license" }, { "n", OPT_BOOL, {(void *)&no_launch }, "enable no-launch mode" }, { "d", 0, {(void*)opt_debug}, "enable debug mode" }, { "f", HAS_ARG | OPT_STRING, {(void*)&config_filename }, "use configfile instead of /etc/ffserver.conf", "configfile" }, -- 2.34.1