[FFmpeg-devel] [PATCH 9/9] dnxhddec: use auto as default idct

Christophe Gisquet christophe.gisquet at gmail.com
Thu Oct 8 08:22:56 CEST 2015


It's an intra codec, so no risk of drift. The simple idct version
clips to an incorrect range, which should be [4;1019] for YUV 10b,
while RGB 10b is fine.
---
 libavcodec/dnxhddec.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c
index 3b1fbdb..1ca867a 100644
--- a/libavcodec/dnxhddec.c
+++ b/libavcodec/dnxhddec.c
@@ -688,6 +688,11 @@ static av_cold int dnxhd_decode_close(AVCodecContext *avctx)
     return 0;
 }
 
+static const AVCodecDefault dnxhd_defaults[] = {
+    { "idct", "auto" },
+    { NULL },
+};
+
 AVCodec ff_dnxhd_decoder = {
     .name           = "dnxhd",
     .long_name      = NULL_IF_CONFIG_SMALL("VC3/DNxHD"),
@@ -700,4 +705,5 @@ AVCodec ff_dnxhd_decoder = {
     .capabilities   = AV_CODEC_CAP_DR1 | AV_CODEC_CAP_FRAME_THREADS |
                       AV_CODEC_CAP_SLICE_THREADS,
     .init_thread_copy = ONLY_IF_THREADS_ENABLED(dnxhd_decode_init_thread_copy),
+    .defaults       = dnxhd_defaults,
 };
-- 
2.6.0



More information about the ffmpeg-devel mailing list