| Message ID | 20231114172051.13872-4-michael@niedermayer.cc |
|---|---|
| State | New |
| Headers | show |
| Series | [FFmpeg-devel,1/6] avcodec/h264: Seperate SEI and IDR recovery handling | expand |
| Context | Check | Description |
|---|---|---|
| yinshiyou/make_loongarch64 | success | Make finished |
| yinshiyou/make_fate_loongarch64 | success | Make fate finished |
| andriy/make_x86 | success | Make finished |
| andriy/make_fate_x86 | success | Make fate finished |
On Tue, Nov 14, 2023 at 6:21 PM Michael Niedermayer <michael@niedermayer.cc> wrote: > > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> > --- > libavcodec/h264_picture.c | 1 + > libavcodec/h264_slice.c | 19 ++++++++++++++++++- > libavcodec/h264dec.c | 1 + > libavcodec/h264dec.h | 4 ++++ > 4 files changed, 24 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/h264_picture.c b/libavcodec/h264_picture.c > index 691c61eea23..3234141dbd6 100644 > --- a/libavcodec/h264_picture.c > +++ b/libavcodec/h264_picture.c > @@ -96,6 +96,7 @@ static void h264_copy_picture_params(H264Picture *dst, const H264Picture *src) > dst->field_picture = src->field_picture; > dst->reference = src->reference; > dst->recovered = src->recovered; > + dst->gray = src->gray; > dst->invalid_gap = src->invalid_gap; > dst->sei_recovery_frame_cnt = src->sei_recovery_frame_cnt; > dst->mb_width = src->mb_width; > diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c > index 4861a2cabba..2c4ab89dae1 100644 > --- a/libavcodec/h264_slice.c > +++ b/libavcodec/h264_slice.c > @@ -457,6 +457,7 @@ int ff_h264_update_thread_context(AVCodecContext *dst, > h->poc.prev_frame_num = h->poc.frame_num; > > h->recovery_frame = h1->recovery_frame; > + h->non_gray = h1->non_gray; > > return err; > } > @@ -1544,11 +1545,14 @@ static int h264_field_start(H264Context *h, const H264SliceContext *sl, > if (ret < 0) > return ret; > h->short_ref[0]->poc = prev->poc + 2U; > + h->short_ref[0]->gray = prev->gray; > ff_thread_report_progress(&h->short_ref[0]->tf, INT_MAX, 0); > if (h->short_ref[0]->field_picture) > ff_thread_report_progress(&h->short_ref[0]->tf, INT_MAX, 1); > - } else if (!h->frame_recovered && !h->avctx->hwaccel) > + } else if (!h->frame_recovered && !h->avctx->hwaccel) { > color_frame(h->short_ref[0]->f, c); > + h->short_ref[0]->gray = 1; > + } While we can't color invalid frames for hwaccels easily, the flag should perhaps still be applied, as they are equally invalid. Thinking about this, maybe the name should be less the color we happened to use for it, and more like "placeholder" or "invalid" or anything like that? > h->short_ref[0]->frame_num = h->poc.prev_frame_num; > } > } > @@ -2007,6 +2011,19 @@ static int h264_slice_init(H264Context *h, H264SliceContext *sl, > (sl->ref_list[j][i].reference & 3); > } > > + if (sl->slice_type_nos == AV_PICTURE_TYPE_I) { > + h->cur_pic_ptr->gray = 0; > + h->non_gray = 1; > + } else { > + int gray = 0; > + for (j = 0; j < sl->list_count; j++) { > + for (i = 0; i < sl->ref_count[j]; i++) { > + gray |= sl->ref_list[j][i].parent->gray; > + } > + } > + h->cur_pic_ptr->gray = gray; > + } > + > if (h->avctx->debug & FF_DEBUG_PICT_INFO) { > av_log(h->avctx, AV_LOG_DEBUG, > "slice:%d %c mb:%d %c%s%s frame:%d poc:%d/%d ref:%d/%d qp:%d loop:%d:%d:%d weight:%d%s %s\n", > diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c > index ef9e60bbf33..7ea55db75dd 100644 > --- a/libavcodec/h264dec.c > +++ b/libavcodec/h264dec.c > @@ -492,6 +492,7 @@ static void h264_decode_flush(AVCodecContext *avctx) > ff_h264_unref_picture(&h->cur_pic); > > h->mb_y = 0; > + h->non_gray = 0; > > ff_h264_free_tables(h); > h->context_initialized = 0; > diff --git a/libavcodec/h264dec.h b/libavcodec/h264dec.h > index ede51951727..366626c056c 100644 > --- a/libavcodec/h264dec.h > +++ b/libavcodec/h264dec.h > @@ -154,6 +154,8 @@ typedef struct H264Picture { > > /// RefStruct reference; its pointee is shared between decoding threads. > atomic_int *decode_error_flags; > + > + int gray; > } H264Picture; > > typedef struct H264Ref { > @@ -567,6 +569,8 @@ typedef struct H264Context { > struct FFRefStructPool *ref_index_pool; > struct FFRefStructPool *decode_error_flags_pool; > int ref2frm[MAX_SLICES][2][64]; ///< reference to frame number lists, used in the loop filter, the first 2 are for -2,-1 > + > + int non_gray; ///< Did we encounter a intra frame after a gray gap frame > } H264Context; > > extern const uint16_t ff_h264_mb_sizes[4]; > -- > 2.17.1 > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
On Tue, Nov 14, 2023 at 06:32:19PM +0100, Hendrik Leppkes wrote: > On Tue, Nov 14, 2023 at 6:21 PM Michael Niedermayer > <michael@niedermayer.cc> wrote: > > > > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> > > --- > > libavcodec/h264_picture.c | 1 + > > libavcodec/h264_slice.c | 19 ++++++++++++++++++- > > libavcodec/h264dec.c | 1 + > > libavcodec/h264dec.h | 4 ++++ > > 4 files changed, 24 insertions(+), 1 deletion(-) > > > > diff --git a/libavcodec/h264_picture.c b/libavcodec/h264_picture.c > > index 691c61eea23..3234141dbd6 100644 > > --- a/libavcodec/h264_picture.c > > +++ b/libavcodec/h264_picture.c > > @@ -96,6 +96,7 @@ static void h264_copy_picture_params(H264Picture *dst, const H264Picture *src) > > dst->field_picture = src->field_picture; > > dst->reference = src->reference; > > dst->recovered = src->recovered; > > + dst->gray = src->gray; > > dst->invalid_gap = src->invalid_gap; > > dst->sei_recovery_frame_cnt = src->sei_recovery_frame_cnt; > > dst->mb_width = src->mb_width; > > diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c > > index 4861a2cabba..2c4ab89dae1 100644 > > --- a/libavcodec/h264_slice.c > > +++ b/libavcodec/h264_slice.c > > @@ -457,6 +457,7 @@ int ff_h264_update_thread_context(AVCodecContext *dst, > > h->poc.prev_frame_num = h->poc.frame_num; > > > > h->recovery_frame = h1->recovery_frame; > > + h->non_gray = h1->non_gray; > > > > return err; > > } > > @@ -1544,11 +1545,14 @@ static int h264_field_start(H264Context *h, const H264SliceContext *sl, > > if (ret < 0) > > return ret; > > h->short_ref[0]->poc = prev->poc + 2U; > > + h->short_ref[0]->gray = prev->gray; > > ff_thread_report_progress(&h->short_ref[0]->tf, INT_MAX, 0); > > if (h->short_ref[0]->field_picture) > > ff_thread_report_progress(&h->short_ref[0]->tf, INT_MAX, 1); > > - } else if (!h->frame_recovered && !h->avctx->hwaccel) > > + } else if (!h->frame_recovered && !h->avctx->hwaccel) { > > color_frame(h->short_ref[0]->f, c); > > + h->short_ref[0]->gray = 1; > > + } > > While we can't color invalid frames for hwaccels easily, the flag > should perhaps still be applied, as they are equally invalid. ok > Thinking about this, maybe the name should be less the color we > happened to use for it, and more like "placeholder" or "invalid" or > anything like that? "invalid" is a quite generic term that covers more than this case. and iam not sure if "placeholder" is really good description of them. I picked "gray" because i had no better idea and they are gray thx [...]
On Tue, Nov 14, 2023 at 07:46:16PM +0100, Michael Niedermayer wrote: > On Tue, Nov 14, 2023 at 06:32:19PM +0100, Hendrik Leppkes wrote: > > On Tue, Nov 14, 2023 at 6:21 PM Michael Niedermayer > > <michael@niedermayer.cc> wrote: > > > > > > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> > > > --- > > > libavcodec/h264_picture.c | 1 + > > > libavcodec/h264_slice.c | 19 ++++++++++++++++++- > > > libavcodec/h264dec.c | 1 + > > > libavcodec/h264dec.h | 4 ++++ > > > 4 files changed, 24 insertions(+), 1 deletion(-) > > > > > > diff --git a/libavcodec/h264_picture.c b/libavcodec/h264_picture.c > > > index 691c61eea23..3234141dbd6 100644 > > > --- a/libavcodec/h264_picture.c > > > +++ b/libavcodec/h264_picture.c > > > @@ -96,6 +96,7 @@ static void h264_copy_picture_params(H264Picture *dst, const H264Picture *src) > > > dst->field_picture = src->field_picture; > > > dst->reference = src->reference; > > > dst->recovered = src->recovered; > > > + dst->gray = src->gray; > > > dst->invalid_gap = src->invalid_gap; > > > dst->sei_recovery_frame_cnt = src->sei_recovery_frame_cnt; > > > dst->mb_width = src->mb_width; > > > diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c > > > index 4861a2cabba..2c4ab89dae1 100644 > > > --- a/libavcodec/h264_slice.c > > > +++ b/libavcodec/h264_slice.c > > > @@ -457,6 +457,7 @@ int ff_h264_update_thread_context(AVCodecContext *dst, > > > h->poc.prev_frame_num = h->poc.frame_num; > > > > > > h->recovery_frame = h1->recovery_frame; > > > + h->non_gray = h1->non_gray; > > > > > > return err; > > > } > > > @@ -1544,11 +1545,14 @@ static int h264_field_start(H264Context *h, const H264SliceContext *sl, > > > if (ret < 0) > > > return ret; > > > h->short_ref[0]->poc = prev->poc + 2U; > > > + h->short_ref[0]->gray = prev->gray; > > > ff_thread_report_progress(&h->short_ref[0]->tf, INT_MAX, 0); > > > if (h->short_ref[0]->field_picture) > > > ff_thread_report_progress(&h->short_ref[0]->tf, INT_MAX, 1); > > > - } else if (!h->frame_recovered && !h->avctx->hwaccel) > > > + } else if (!h->frame_recovered && !h->avctx->hwaccel) { > > > color_frame(h->short_ref[0]->f, c); > > > + h->short_ref[0]->gray = 1; > > > + } > > > > While we can't color invalid frames for hwaccels easily, the flag > > should perhaps still be applied, as they are equally invalid. > > ok will apply with this changed > > > > Thinking about this, maybe the name should be less the color we > > happened to use for it, and more like "placeholder" or "invalid" or > > anything like that? > > "invalid" is a quite generic term that covers more than this case. > and iam not sure if "placeholder" is really good description of them. > I picked "gray" because i had no better idea and they are gray if someone comes up with a better name, we can rename it but gray is kind of fitting thx [...]
On Sun, Nov 19, 2023 at 7:11 PM Michael Niedermayer <michael@niedermayer.cc> wrote: > On Tue, Nov 14, 2023 at 07:46:16PM +0100, Michael Niedermayer wrote: > > On Tue, Nov 14, 2023 at 06:32:19PM +0100, Hendrik Leppkes wrote: > > > On Tue, Nov 14, 2023 at 6:21 PM Michael Niedermayer > > > <michael@niedermayer.cc> wrote: > > > > > > > > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> > > > > --- > > > > libavcodec/h264_picture.c | 1 + > > > > libavcodec/h264_slice.c | 19 ++++++++++++++++++- > > > > libavcodec/h264dec.c | 1 + > > > > libavcodec/h264dec.h | 4 ++++ > > > > 4 files changed, 24 insertions(+), 1 deletion(-) > > > > > > > > diff --git a/libavcodec/h264_picture.c b/libavcodec/h264_picture.c > > > > index 691c61eea23..3234141dbd6 100644 > > > > --- a/libavcodec/h264_picture.c > > > > +++ b/libavcodec/h264_picture.c > > > > @@ -96,6 +96,7 @@ static void h264_copy_picture_params(H264Picture > *dst, const H264Picture *src) > > > > dst->field_picture = src->field_picture; > > > > dst->reference = src->reference; > > > > dst->recovered = src->recovered; > > > > + dst->gray = src->gray; > > > > dst->invalid_gap = src->invalid_gap; > > > > dst->sei_recovery_frame_cnt = src->sei_recovery_frame_cnt; > > > > dst->mb_width = src->mb_width; > > > > diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c > > > > index 4861a2cabba..2c4ab89dae1 100644 > > > > --- a/libavcodec/h264_slice.c > > > > +++ b/libavcodec/h264_slice.c > > > > @@ -457,6 +457,7 @@ int ff_h264_update_thread_context(AVCodecContext > *dst, > > > > h->poc.prev_frame_num = h->poc.frame_num; > > > > > > > > h->recovery_frame = h1->recovery_frame; > > > > + h->non_gray = h1->non_gray; > > > > > > > > return err; > > > > } > > > > @@ -1544,11 +1545,14 @@ static int h264_field_start(H264Context *h, > const H264SliceContext *sl, > > > > if (ret < 0) > > > > return ret; > > > > h->short_ref[0]->poc = prev->poc + 2U; > > > > + h->short_ref[0]->gray = prev->gray; > > > > ff_thread_report_progress(&h->short_ref[0]->tf, > INT_MAX, 0); > > > > if (h->short_ref[0]->field_picture) > > > > ff_thread_report_progress(&h->short_ref[0]->tf, > INT_MAX, 1); > > > > - } else if (!h->frame_recovered && !h->avctx->hwaccel) > > > > + } else if (!h->frame_recovered && !h->avctx->hwaccel) { > > > > color_frame(h->short_ref[0]->f, c); > > > > + h->short_ref[0]->gray = 1; > > > > + } > > > > > > While we can't color invalid frames for hwaccels easily, the flag > > > should perhaps still be applied, as they are equally invalid. > > > > ok > > will apply with this changed > > > > > > > > Thinking about this, maybe the name should be less the color we > > > happened to use for it, and more like "placeholder" or "invalid" or > > > anything like that? > > > > "invalid" is a quite generic term that covers more than this case. > > and iam not sure if "placeholder" is really good description of them. > > I picked "gray" because i had no better idea and they are gray > > if someone comes up with a better name, we can rename it > but gray is kind of fitting > Sorry, what exactly are h264 gray frames? Could you add some documentation to the new field in H264Picture too?
On Sun, Nov 19, 2023 at 10:01:23PM -0500, Vittorio Giovara wrote: > On Sun, Nov 19, 2023 at 7:11 PM Michael Niedermayer <michael@niedermayer.cc> > wrote: > > > On Tue, Nov 14, 2023 at 07:46:16PM +0100, Michael Niedermayer wrote: > > > On Tue, Nov 14, 2023 at 06:32:19PM +0100, Hendrik Leppkes wrote: > > > > On Tue, Nov 14, 2023 at 6:21 PM Michael Niedermayer > > > > <michael@niedermayer.cc> wrote: > > > > > > > > > > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> > > > > > --- > > > > > libavcodec/h264_picture.c | 1 + > > > > > libavcodec/h264_slice.c | 19 ++++++++++++++++++- > > > > > libavcodec/h264dec.c | 1 + > > > > > libavcodec/h264dec.h | 4 ++++ > > > > > 4 files changed, 24 insertions(+), 1 deletion(-) > > > > > > > > > > diff --git a/libavcodec/h264_picture.c b/libavcodec/h264_picture.c > > > > > index 691c61eea23..3234141dbd6 100644 > > > > > --- a/libavcodec/h264_picture.c > > > > > +++ b/libavcodec/h264_picture.c > > > > > @@ -96,6 +96,7 @@ static void h264_copy_picture_params(H264Picture > > *dst, const H264Picture *src) > > > > > dst->field_picture = src->field_picture; > > > > > dst->reference = src->reference; > > > > > dst->recovered = src->recovered; > > > > > + dst->gray = src->gray; > > > > > dst->invalid_gap = src->invalid_gap; > > > > > dst->sei_recovery_frame_cnt = src->sei_recovery_frame_cnt; > > > > > dst->mb_width = src->mb_width; > > > > > diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c > > > > > index 4861a2cabba..2c4ab89dae1 100644 > > > > > --- a/libavcodec/h264_slice.c > > > > > +++ b/libavcodec/h264_slice.c > > > > > @@ -457,6 +457,7 @@ int ff_h264_update_thread_context(AVCodecContext > > *dst, > > > > > h->poc.prev_frame_num = h->poc.frame_num; > > > > > > > > > > h->recovery_frame = h1->recovery_frame; > > > > > + h->non_gray = h1->non_gray; > > > > > > > > > > return err; > > > > > } > > > > > @@ -1544,11 +1545,14 @@ static int h264_field_start(H264Context *h, > > const H264SliceContext *sl, > > > > > if (ret < 0) > > > > > return ret; > > > > > h->short_ref[0]->poc = prev->poc + 2U; > > > > > + h->short_ref[0]->gray = prev->gray; > > > > > ff_thread_report_progress(&h->short_ref[0]->tf, > > INT_MAX, 0); > > > > > if (h->short_ref[0]->field_picture) > > > > > ff_thread_report_progress(&h->short_ref[0]->tf, > > INT_MAX, 1); > > > > > - } else if (!h->frame_recovered && !h->avctx->hwaccel) > > > > > + } else if (!h->frame_recovered && !h->avctx->hwaccel) { > > > > > color_frame(h->short_ref[0]->f, c); > > > > > + h->short_ref[0]->gray = 1; > > > > > + } > > > > > > > > While we can't color invalid frames for hwaccels easily, the flag > > > > should perhaps still be applied, as they are equally invalid. > > > > > > ok > > > > will apply with this changed > > > > > > > > > > > > Thinking about this, maybe the name should be less the color we > > > > happened to use for it, and more like "placeholder" or "invalid" or > > > > anything like that? > > > > > > "invalid" is a quite generic term that covers more than this case. > > > and iam not sure if "placeholder" is really good description of them. > > > I picked "gray" because i had no better idea and they are gray > > > > if someone comes up with a better name, we can rename it > > but gray is kind of fitting > > > > Sorry, what exactly are h264 gray frames? Bascically when we have "missing" frames we fill them by copying a frame close by or if we have none then we fill them with gray. (this can occur due to lost frames or at the stream start before sei_recovery_frame_cnt) These frames that we fill with gray are gray frames Before this patchset we did not keep track of which frames where gray so they could not be avoided as reference frames later. We only avoided them during their creation. After this patchset by default we avoid using gray frames as reference which simply results in better looking frames. Also i have a stream that is a classic IBBPBBP style stream but the B frames use past B frames as references and accross intra frame so that if gray frames are not avoided B frames will forever be corrupt with gray blocks even after sei_recovery_frame_cnt. Other H264 decoders do not show this behaviour that we have prior to this patchset, they avoid these empty/gray frames. Iam waiting for the owner of this stream to allow me to publish it then ill make a fate test with it. Basically this patchset makes things look better in every case where it looks different than before that i found. Also you can tune the options to get the old behavior or to drop affected frames instead of adjusting references away from gray frames > > Could you add some documentation to the new field in H264Picture too? ill post a patch documenting that thx [...]
diff --git a/libavcodec/h264_picture.c b/libavcodec/h264_picture.c index 691c61eea23..3234141dbd6 100644 --- a/libavcodec/h264_picture.c +++ b/libavcodec/h264_picture.c @@ -96,6 +96,7 @@ static void h264_copy_picture_params(H264Picture *dst, const H264Picture *src) dst->field_picture = src->field_picture; dst->reference = src->reference; dst->recovered = src->recovered; + dst->gray = src->gray; dst->invalid_gap = src->invalid_gap; dst->sei_recovery_frame_cnt = src->sei_recovery_frame_cnt; dst->mb_width = src->mb_width; diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c index 4861a2cabba..2c4ab89dae1 100644 --- a/libavcodec/h264_slice.c +++ b/libavcodec/h264_slice.c @@ -457,6 +457,7 @@ int ff_h264_update_thread_context(AVCodecContext *dst, h->poc.prev_frame_num = h->poc.frame_num; h->recovery_frame = h1->recovery_frame; + h->non_gray = h1->non_gray; return err; } @@ -1544,11 +1545,14 @@ static int h264_field_start(H264Context *h, const H264SliceContext *sl, if (ret < 0) return ret; h->short_ref[0]->poc = prev->poc + 2U; + h->short_ref[0]->gray = prev->gray; ff_thread_report_progress(&h->short_ref[0]->tf, INT_MAX, 0); if (h->short_ref[0]->field_picture) ff_thread_report_progress(&h->short_ref[0]->tf, INT_MAX, 1); - } else if (!h->frame_recovered && !h->avctx->hwaccel) + } else if (!h->frame_recovered && !h->avctx->hwaccel) { color_frame(h->short_ref[0]->f, c); + h->short_ref[0]->gray = 1; + } h->short_ref[0]->frame_num = h->poc.prev_frame_num; } } @@ -2007,6 +2011,19 @@ static int h264_slice_init(H264Context *h, H264SliceContext *sl, (sl->ref_list[j][i].reference & 3); } + if (sl->slice_type_nos == AV_PICTURE_TYPE_I) { + h->cur_pic_ptr->gray = 0; + h->non_gray = 1; + } else { + int gray = 0; + for (j = 0; j < sl->list_count; j++) { + for (i = 0; i < sl->ref_count[j]; i++) { + gray |= sl->ref_list[j][i].parent->gray; + } + } + h->cur_pic_ptr->gray = gray; + } + if (h->avctx->debug & FF_DEBUG_PICT_INFO) { av_log(h->avctx, AV_LOG_DEBUG, "slice:%d %c mb:%d %c%s%s frame:%d poc:%d/%d ref:%d/%d qp:%d loop:%d:%d:%d weight:%d%s %s\n", diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c index ef9e60bbf33..7ea55db75dd 100644 --- a/libavcodec/h264dec.c +++ b/libavcodec/h264dec.c @@ -492,6 +492,7 @@ static void h264_decode_flush(AVCodecContext *avctx) ff_h264_unref_picture(&h->cur_pic); h->mb_y = 0; + h->non_gray = 0; ff_h264_free_tables(h); h->context_initialized = 0; diff --git a/libavcodec/h264dec.h b/libavcodec/h264dec.h index ede51951727..366626c056c 100644 --- a/libavcodec/h264dec.h +++ b/libavcodec/h264dec.h @@ -154,6 +154,8 @@ typedef struct H264Picture { /// RefStruct reference; its pointee is shared between decoding threads. atomic_int *decode_error_flags; + + int gray; } H264Picture; typedef struct H264Ref { @@ -567,6 +569,8 @@ typedef struct H264Context { struct FFRefStructPool *ref_index_pool; struct FFRefStructPool *decode_error_flags_pool; int ref2frm[MAX_SLICES][2][64]; ///< reference to frame number lists, used in the loop filter, the first 2 are for -2,-1 + + int non_gray; ///< Did we encounter a intra frame after a gray gap frame } H264Context; extern const uint16_t ff_h264_mb_sizes[4];
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> --- libavcodec/h264_picture.c | 1 + libavcodec/h264_slice.c | 19 ++++++++++++++++++- libavcodec/h264dec.c | 1 + libavcodec/h264dec.h | 4 ++++ 4 files changed, 24 insertions(+), 1 deletion(-)