From e397c4cea10938c6699283179a6611098e4af089 Mon Sep 17 00:00:00 2001 From: Guillaume Desmottes Date: Mon, 25 Mar 2019 13:49:39 +0100 Subject: [PATCH] video-info: explain in logs why colorimetry are rejected It wasn't clear from the logs why some colorimetry format were rejected. --- gst-libs/gst/video/video-info.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/gst-libs/gst/video/video-info.c b/gst-libs/gst/video/video-info.c index 033df01..ed10887 100644 --- a/gst-libs/gst/video/video-info.c +++ b/gst-libs/gst/video/video-info.c @@ -185,12 +185,19 @@ validate_colorimetry (GstVideoInfo * info) const GstVideoFormatInfo *finfo = info->finfo; if (!GST_VIDEO_FORMAT_INFO_IS_RGB (finfo) && - info->colorimetry.matrix == GST_VIDEO_COLOR_MATRIX_RGB) + info->colorimetry.matrix == GST_VIDEO_COLOR_MATRIX_RGB) { + GST_WARNING + ("color matrix RGB is only supported with RGB format, %s is not", + finfo->name); return FALSE; + } if (GST_VIDEO_FORMAT_INFO_IS_YUV (finfo) && - info->colorimetry.matrix == GST_VIDEO_COLOR_MATRIX_UNKNOWN) + info->colorimetry.matrix == GST_VIDEO_COLOR_MATRIX_UNKNOWN) { + GST_WARNING ("Need to specify a color matrix when using YUV format (%s)", + finfo->name); return FALSE; + } return TRUE; } -- 2.7.4