Hi!

First time poster so please excuse any mistakes made. I've tried submitting
a patch through git send-email but got issues authentication through gmail.
Attached is a patch file created with git format-patch.

Commit message:

Added support for version field flag 0x04 in OpenEXR files regarding long
names in attributes.
Should have no impact on decoding of images.

Also added a bit more verbose logging of OpenEXR 2.x flags not yet
supported.

-- 
-Daniel
From 0a8c7b74ef6ad39c7ce4a2eaa8327dd4a1f9d851 Mon Sep 17 00:00:00 2001
From: Daniel Flehner Heen <flehnerhee...@gmail.com>
Date: Tue, 20 Feb 2018 23:36:29 +0100
Subject: [PATCH] Added support for version field flag 0x04 in OpenEXR files

Added support for version field flag 0x04 in OpenEXR files regarding long names in attributes.
Should have no impact on decoding of images.

Also added a bit more verbose logging of OpenEXR 2.x flags not yet supported.
---
 libavcodec/exr.c | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/libavcodec/exr.c b/libavcodec/exr.c
index 47f59bd..238ee4a 100644
--- a/libavcodec/exr.c
+++ b/libavcodec/exr.c
@@ -1354,7 +1354,15 @@ static int decode_header(EXRContext *s, AVFrame *frame)
         s->is_tile = 0;
     else if (flags & 0x02)
         s->is_tile = 1;
-    else{
+    else if (flags & 0x04)
+        av_log(s->avctx, AV_LOG_DEBUG, "Long attribute names flag detected %d.\n", flags);
+    else if (flags & 0x08) {
+        avpriv_report_missing_feature(s->avctx, "OpenEXR 2.x deep format flag %d", flags);
+        return AVERROR_PATCHWELCOME;
+    } else if (flags & 0x10) {
+        avpriv_report_missing_feature(s->avctx, "OpenEXR 2.x multipart flag %d", flags);
+        return AVERROR_PATCHWELCOME;
+    } else {
         avpriv_report_missing_feature(s->avctx, "flags %d", flags);
         return AVERROR_PATCHWELCOME;
     }
-- 
2.7.4

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

Reply via email to