diff mbox series

[FFmpeg-devel,14/14] h264dec: do not set picture_structure on init

Message ID 20200327125747.13460-14-anton@khirnov.net
State Accepted
Headers show
Series [FFmpeg-devel,01/14] mpeg4videodec: do not copy a range of fields at once
Related show

Checks

Context Check Description
andriy/ffmpeg-patchwork pending
andriy/ffmpeg-patchwork success Applied patch
andriy/ffmpeg-patchwork success Configure finished
andriy/ffmpeg-patchwork success Make finished
andriy/ffmpeg-patchwork success Make fate finished

Commit Message

Anton Khirnov March 27, 2020, 12:57 p.m. UTC
This has been cargo culted from mpegvideo and serves no useful purpose.
It will be initialize correctly in h264_field_start()
---
 libavcodec/h264dec.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Michael Niedermayer April 6, 2020, 10:59 p.m. UTC | #1
On Fri, Mar 27, 2020 at 01:57:48PM +0100, Anton Khirnov wrote:
> This has been cargo culted from mpegvideo and serves no useful purpose.
> It will be initialize correctly in h264_field_start()
> ---
>  libavcodec/h264dec.c | 1 -
>  1 file changed, 1 deletion(-)

LGTM

thx

[...]
diff mbox series

Patch

diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c
index 7679567548..57f4fec3ca 100644
--- a/libavcodec/h264dec.c
+++ b/libavcodec/h264dec.c
@@ -309,7 +309,6 @@  static int h264_init_context(AVCodecContext *avctx, H264Context *h)
     h->width_from_caller     = avctx->width;
     h->height_from_caller    = avctx->height;
 
-    h->picture_structure     = PICT_FRAME;
     h->workaround_bugs       = avctx->workaround_bugs;
     h->flags                 = avctx->flags;
     h->poc.prev_poc_msb      = 1 << 16;