On Fri, 13 Jul 2018, Carl Eugen Hoyos wrote:

2018-07-02 23:53 GMT+02:00, Marton Balint <c...@passwd.hu>:
SDL from version 2.0.8 has support for full range YUV and specifying
BT601/BT709 color space for YUV->RGB conversion.

Signed-off-by: Marton Balint <c...@passwd.hu>
---
 fftools/ffplay.c | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/fftools/ffplay.c b/fftools/ffplay.c
index 55cea32cae..368e262123 100644
--- a/fftools/ffplay.c
+++ b/fftools/ffplay.c
@@ -954,6 +954,22 @@ static int upload_texture(SDL_Texture **tex, AVFrame
*frame, struct SwsContext *
     return ret;
 }

+static void set_sdl_yuv_conversion_mode(AVFrame *frame)
+{
+#if SDL_VERSION_ATLEAST(2,0,8)
+    SDL_YUV_CONVERSION_MODE mode = SDL_YUV_CONVERSION_AUTOMATIC;
+    if (frame && (frame->format == AV_PIX_FMT_YUV420P || frame->format ==
AV_PIX_FMT_YUYV422 || frame->format == AV_PIX_FMT_UYVY422)) {

What happens if the format is YUVJ420P?

That cannot happen, because the filtering code would automagically convert it to one of the supported texture formats (typically YUV420P). In theory we could add support for direct rendering YUVJ420P (without software conversion), but since that format is deprecated, I'd rather not clutter the code with it.

Regards,
Marton
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to