Browse Source

dnxhddec: Introduce DNXHD_VARIABLE

Currently not used, but will be used to indicate that a CIDEntry field
is not set, because it is variable, and that checks should be adapted.

Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
tags/n3.0
Christophe Gisquet Michael Niedermayer 9 years ago
parent
commit
1f4ff53aea
2 changed files with 7 additions and 2 deletions
  1. +3
    -0
      libavcodec/dnxhddata.h
  2. +4
    -2
      libavcodec/dnxhddec.c

+ 3
- 0
libavcodec/dnxhddata.h View File

@@ -30,6 +30,9 @@
#define DNXHD_INTERLACED (1<<0)
#define DNXHD_444 (1<<1)

/** Indicate that a CIDEntry value must be read in the bitstream */
#define DNXHD_VARIABLE 0

typedef struct CIDEntry {
int cid;
unsigned int width, height;


+ 4
- 2
libavcodec/dnxhddec.c View File

@@ -106,7 +106,8 @@ static int dnxhd_init_vlc(DNXHDContext *ctx, uint32_t cid)
av_log(ctx->avctx, AV_LOG_ERROR, "unsupported cid %d\n", cid);
return AVERROR(ENOSYS);
}
if (ff_dnxhd_cid_table[index].bit_depth != ctx->bit_depth) {
if (ff_dnxhd_cid_table[index].bit_depth != ctx->bit_depth &&
ff_dnxhd_cid_table[index].bit_depth != DNXHD_VARIABLE) {
av_log(ctx->avctx, AV_LOG_ERROR, "bit depth mismatches %d %d\n", ff_dnxhd_cid_table[index].bit_depth, ctx->bit_depth);
return AVERROR_INVALIDDATA;
}
@@ -228,7 +229,8 @@ static int dnxhd_decode_header(DNXHDContext *ctx, AVFrame *frame,

// make sure profile size constraints are respected
// DNx100 allows 1920->1440 and 1280->960 subsampling
if (ctx->width != ctx->cid_table->width) {
if (ctx->width != ctx->cid_table->width &&
ctx->cid_table->width != DNXHD_VARIABLE) {
av_reduce(&ctx->avctx->sample_aspect_ratio.num,
&ctx->avctx->sample_aspect_ratio.den,
ctx->width, ctx->cid_table->width, 255);


Loading…
Cancel
Save