summaryrefslogtreecommitdiff
path: root/fftools/ffplay.c
diff options
context:
space:
mode:
authorAnton Khirnov <anton@khirnov.net>2023-07-14 13:47:23 +0200
committerAnton Khirnov <anton@khirnov.net>2023-07-20 20:47:40 +0200
commit9cb47c78d66c0d8abf5dbedd254d3761d5c5b06a (patch)
tree1cb27929156e937ac021d69574f7203f29fd168c /fftools/ffplay.c
parent4adbe2d219c13d5342d073aace5001a64c57d9c8 (diff)
fftools: return errors from parse_number_or_die() instead of aborting
Rename the function to just parse_number().
Diffstat (limited to 'fftools/ffplay.c')
-rw-r--r--fftools/ffplay.c25
1 files changed, 21 insertions, 4 deletions
diff --git a/fftools/ffplay.c b/fftools/ffplay.c
index 1f9757d607..6ca1ad98bf 100644
--- a/fftools/ffplay.c
+++ b/fftools/ffplay.c
@@ -3440,13 +3440,23 @@ static void event_loop(VideoState *cur_stream)
static int opt_width(void *optctx, const char *opt, const char *arg)
{
- screen_width = parse_number_or_die(opt, arg, OPT_INT64, 1, INT_MAX);
+ double num;
+ int ret = parse_number(opt, arg, OPT_INT64, 1, INT_MAX, &num);
+ if (ret < 0)
+ return ret;
+
+ screen_width = num;
return 0;
}
static int opt_height(void *optctx, const char *opt, const char *arg)
{
- screen_height = parse_number_or_die(opt, arg, OPT_INT64, 1, INT_MAX);
+ double num;
+ int ret = parse_number(opt, arg, OPT_INT64, 1, INT_MAX, &num);
+ if (ret < 0)
+ return ret;
+
+ screen_height = num;
return 0;
}
@@ -3491,8 +3501,15 @@ static int opt_show_mode(void *optctx, const char *opt, const char *arg)
{
show_mode = !strcmp(arg, "video") ? SHOW_MODE_VIDEO :
!strcmp(arg, "waves") ? SHOW_MODE_WAVES :
- !strcmp(arg, "rdft" ) ? SHOW_MODE_RDFT :
- parse_number_or_die(opt, arg, OPT_INT, 0, SHOW_MODE_NB-1);
+ !strcmp(arg, "rdft" ) ? SHOW_MODE_RDFT : SHOW_MODE_NONE;
+
+ if (show_mode == SHOW_MODE_NONE) {
+ double num;
+ int ret = parse_number(opt, arg, OPT_INT, 0, SHOW_MODE_NB-1, &num);
+ if (ret < 0)
+ return ret;
+ show_mode = num;
+ }
return 0;
}