]> granicus.if.org Git - handbrake/commitdiff
contrib: Add libav colr patch to support new color signaling in mov.
authorBradley Sepos <bradley@bradleysepos.com>
Fri, 25 Nov 2016 14:39:51 +0000 (09:39 -0500)
committerJohn Stebbins <jstebbins.hb@gmail.com>
Sat, 17 Dec 2016 15:28:51 +0000 (07:28 -0800)
Needed for BT.2020, etc.

contrib/ffmpeg/A02-mov-colr.patch [new file with mode: 0644]

diff --git a/contrib/ffmpeg/A02-mov-colr.patch b/contrib/ffmpeg/A02-mov-colr.patch
new file mode 100644 (file)
index 0000000..b65b465
--- /dev/null
@@ -0,0 +1,57 @@
+diff --git a/libavformat/mov.c b/libavformat/mov.c
+index 36e75d5..fee9f36 100644
+--- a/libavformat/mov.c
++++ b/libavformat/mov.c
+@@ -36,6 +36,7 @@
+ #include "libavutil/avstring.h"
+ #include "libavutil/dict.h"
+ #include "libavutil/opt.h"
++#include "libavutil/pixdesc.h"
+ #include "libavcodec/ac3tab.h"
+ #include "avformat.h"
+ #include "internal.h"
+@@ -1083,16 +1084,14 @@ static int mov_read_colr(MOVContext *c, AVIOContext *pb, MOVAtom atom)
+             st->codecpar->color_range = AVCOL_RANGE_JPEG;
+         else
+             st->codecpar->color_range = AVCOL_RANGE_MPEG;
+-        /* 14496-12 references JPEG XR specs (rather than the more complete
+-         * 23001-8) so some adjusting is required */
+-        if (color_primaries >= AVCOL_PRI_FILM)
++
++        if (!av_color_primaries_name(color_primaries))
+             color_primaries = AVCOL_PRI_UNSPECIFIED;
+-        if ((color_trc >= AVCOL_TRC_LINEAR &&
+-             color_trc <= AVCOL_TRC_LOG_SQRT) ||
+-            color_trc >= AVCOL_TRC_BT2020_10)
++        if (!av_color_transfer_name(color_trc))
+             color_trc = AVCOL_TRC_UNSPECIFIED;
+-        if (color_matrix >= AVCOL_SPC_BT2020_NCL)
++        if (!av_color_space_name(color_matrix))
+             color_matrix = AVCOL_SPC_UNSPECIFIED;
++
+         st->codecpar->color_primaries = color_primaries;
+         st->codecpar->color_trc       = color_trc;
+         st->codecpar->color_space     = color_matrix;
+@@ -1102,17 +1101,22 @@ static int mov_read_colr(MOVContext *c, AVIOContext *pb, MOVAtom atom)
+         case 1: st->codecpar->color_primaries = AVCOL_PRI_BT709; break;
+         case 5: st->codecpar->color_primaries = AVCOL_PRI_SMPTE170M; break;
+         case 6: st->codecpar->color_primaries = AVCOL_PRI_SMPTE240M; break;
++        case 9: st->codecpar->color_primaries = AVCOL_PRI_BT2020; break;
++        case 10: st->codecpar->color_primaries = AVCOL_PRI_SMPTE431; break;
++        case 11: st->codecpar->color_primaries = AVCOL_PRI_SMPTE432; break;
+         }
+         /* color transfer, Table 4-5 */
+         switch (color_trc) {
+         case 1: st->codecpar->color_trc = AVCOL_TRC_BT709; break;
+         case 7: st->codecpar->color_trc = AVCOL_TRC_SMPTE240M; break;
++        case 17: st->codecpar->color_trc = AVCOL_TRC_SMPTE428; break;
+         }
+         /* color matrix, Table 4-6 */
+         switch (color_matrix) {
+         case 1: st->codecpar->color_space = AVCOL_SPC_BT709; break;
+         case 6: st->codecpar->color_space = AVCOL_SPC_BT470BG; break;
+         case 7: st->codecpar->color_space = AVCOL_SPC_SMPTE240M; break;
++        case 9: st->codecpar->color_space = AVCOL_SPC_BT2020_NCL; break;
+         }
+     }
+     av_log(c->fc, AV_LOG_TRACE, "\n");