summaryrefslogtreecommitdiff
path: root/libavcodec/av1_parser.c
diff options
context:
space:
mode:
authorJames Almer <jamrial@gmail.com>2020-08-23 16:23:57 -0300
committerJames Almer <jamrial@gmail.com>2020-08-23 20:54:15 -0300
commitc8716b5029693f2d02fd2bffe18867c677fa3630 (patch)
tree56f0f7b3294759465b68c40ab2c3b2b0044486a1 /libavcodec/av1_parser.c
parent6c20207dceefa0452c65c719f0326cbc0177e827 (diff)
avcodec/av1_parser: read frame properties directly from AV1RawFrameHeader
Simplifies code Reviewed-by: Mark Thompson <sw@jkqxz.net> Signed-off-by: James Almer <jamrial@gmail.com>
Diffstat (limited to 'libavcodec/av1_parser.c')
-rw-r--r--libavcodec/av1_parser.c28
1 files changed, 6 insertions, 22 deletions
diff --git a/libavcodec/av1_parser.c b/libavcodec/av1_parser.c
index cd426a2b0f..1b9868d364 100644
--- a/libavcodec/av1_parser.c
+++ b/libavcodec/av1_parser.c
@@ -99,7 +99,6 @@ static int av1_parser_parse(AVCodecParserContext *ctx,
CodedBitstreamUnit *unit = &td->units[i];
AV1RawOBU *obu = unit->content;
AV1RawFrameHeader *frame;
- int frame_type;
if (unit->type == AV1_OBU_FRAME)
frame = &obu->obu.frame.header;
@@ -111,30 +110,15 @@ static int av1_parser_parse(AVCodecParserContext *ctx,
if (obu->header.spatial_id > 0)
continue;
- if (frame->show_existing_frame) {
- AV1ReferenceFrameState *ref = &av1->ref[frame->frame_to_show_map_idx];
-
- if (!ref->valid) {
- av_log(avctx, AV_LOG_ERROR, "Invalid reference frame\n");
- goto end;
- }
-
- ctx->width = ref->frame_width;
- ctx->height = ref->frame_height;
- frame_type = ref->frame_type;
-
- ctx->key_frame = 0;
- } else if (!frame->show_frame) {
+ if (!frame->show_frame)
continue;
- } else {
- ctx->width = av1->frame_width;
- ctx->height = av1->frame_height;
- frame_type = frame->frame_type;
- ctx->key_frame = frame_type == AV1_FRAME_KEY;
- }
+ ctx->width = frame->frame_width_minus_1 + 1;
+ ctx->height = frame->frame_height_minus_1 + 1;
+
+ ctx->key_frame = frame->frame_type == AV1_FRAME_KEY && !frame->show_existing_frame;
- switch (frame_type) {
+ switch (frame->frame_type) {
case AV1_FRAME_KEY:
case AV1_FRAME_INTRA_ONLY:
ctx->pict_type = AV_PICTURE_TYPE_I;