> -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of > Andreas Rheinhardt > Sent: 2020年12月31日 7:31 > To: ffmpeg-devel@ffmpeg.org > Cc: Andreas Rheinhardt <andreas.rheinha...@gmail.com> > Subject: [FFmpeg-devel] [PATCH 02/30] avfilter/vf_addroi: Use chars instead of > strings for one-char strings > > Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@gmail.com> > --- > libavfilter/vf_addroi.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/libavfilter/vf_addroi.c b/libavfilter/vf_addroi.c index > 489998ce73..d6b83ee854 100644 > --- a/libavfilter/vf_addroi.c > +++ b/libavfilter/vf_addroi.c > @@ -26,8 +26,8 @@ enum { > X, Y, W, H, > NB_PARAMS, > }; > -static const char *addroi_param_names[] = { > - "x", "y", "w", "h", > +static const char addroi_param_names[] = { > + 'x', 'y', 'w', 'h', > }; > > enum { > @@ -74,12 +74,12 @@ static int addroi_config_input(AVFilterLink *inlink) > > val = av_expr_eval(ctx->region_expr[i], vars, NULL); > if (val < 0.0) { > - av_log(avctx, AV_LOG_WARNING, "Calculated value %g for %s is > " > + av_log(avctx, AV_LOG_WARNING, "Calculated value %g for %c is > " > "less than zero - using zero instead.\n", val, > addroi_param_names[i]); > val = 0.0; > } else if (val > max_value) { > - av_log(avctx, AV_LOG_WARNING, "Calculated value %g for %s is > " > + av_log(avctx, AV_LOG_WARNING, "Calculated value %g for %c is > " > "greater than maximum allowed value %d - " > "using %d instead.\n", val, addroi_param_names[i], > max_value, max_value); @@ -194,7 +194,7 @@ static > av_cold int addroi_init(AVFilterContext *avctx) > 0, avctx); > if (err < 0) { > av_log(ctx, AV_LOG_ERROR, > - "Error parsing %s expression '%s'.\n", > + "Error parsing %c expression '%s'.\n", > addroi_param_names[i], ctx->region_str[i]); > return err; > } > -- this patch LGTM _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".