On 20/10/2016 17:06, Ronald S. Bultje wrote:
--- libavfilter/vf_colorspace.c | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-)diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c index c74fe00..f64163f 100644 --- a/libavfilter/vf_colorspace.c +++ b/libavfilter/vf_colorspace.c @@ -163,6 +163,8 @@ typedef struct ColorSpaceContext { yuv2yuv_fn yuv2yuv; double yuv2rgb_dbl_coeffs[3][3], rgb2yuv_dbl_coeffs[3][3]; int in_y_rng, in_uv_rng, out_y_rng, out_uv_rng; + + int did_range_warn; } ColorSpaceContext; // FIXME deal with odd width/heights (or just forbid it) @@ -523,9 +525,15 @@ static int get_range_off(AVFilterContext *ctx, int *off, enum AVColorRange rng, int depth) { switch (rng) { - case AVCOL_RANGE_UNSPECIFIED: - av_log(ctx, AV_LOG_WARNING, "Input range not set, assuming tv/mpeg\n"); + case AVCOL_RANGE_UNSPECIFIED: { + ColorSpaceContext *s = ctx->priv; + + if (!s->did_range_warn) { + av_log(ctx, AV_LOG_WARNING, "Input range not set, assuming tv/mpeg\n"); + s->did_range_warn = 1; + } // fall-through + } case AVCOL_RANGE_MPEG: *off = 16 << (depth - 8); *y_rng = 219 << (depth - 8);
I think did_warn_range would sound more natural. LGTM otherwise (with or without the change).
-- Josh _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
