Re: [FFmpeg-devel] [PATCH] h264: Support multi-field closed captions by using AVBufferRef and not resetting per field

2018-08-17 Thread Carl Eugen Hoyos
2018-08-16 12:52 GMT+02:00, jos...@ob-encoder.com : > @@ -429,6 +429,12 @@ int ff_h264_update_thread_context(AVCodecContext *dst, > MAX_DELAYED_PIC_COUNT + 2, h, h1); > > h->frame_recovered = h1->frame_recovered; > +if (h1->sei.a53_caption.buf_ref) { > +

Re: [FFmpeg-devel] [PATCH] h264: Support multi-field closed captions by using AVBufferRef and not resetting per field

2018-08-16 Thread James Almer
On 8/16/2018 7:52 AM, jos...@ob-encoder.com wrote: > diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c > index ede9a1a6ea..7bbe65878f 100644 > --- a/libavcodec/h264_slice.c > +++ b/libavcodec/h264_slice.c > @@ -429,6 +429,12 @@ int ff_h264_update_thread_context(AVCodecContext *dst, >

Re: [FFmpeg-devel] [PATCH] h264: Support multi-field closed captions by using AVBufferRef and not resetting per field

2018-08-16 Thread James Almer
On 8/16/2018 7:52 AM, jos...@ob-encoder.com wrote: > From: Kieran Kunhya > > --- > This is also useful when CC is fully contained in the first field. > > libavcodec/h264_sei.c | 15 --- > libavcodec/h264_sei.h | 3 +-- > libavcodec/h264_slice.c | 15 ++- > libavcode

Re: [FFmpeg-devel] [PATCH] h264: Support multi-field closed captions by using AVBufferRef and not resetting per field

2018-08-16 Thread Devin Heitmueller
Hello, > On Aug 16, 2018, at 6:52 AM, jos...@ob-encoder.com wrote: > > From: Kieran Kunhya > > --- > This is also useful when CC is fully contained in the first field. FWIW: I’ve been running a variant of this this patch for a while now. It’s needed for PAFF formatted H.264 streams (i.e. sli

[FFmpeg-devel] [PATCH] h264: Support multi-field closed captions by using AVBufferRef and not resetting per field

2018-08-16 Thread joshdk
From: Kieran Kunhya --- This is also useful when CC is fully contained in the first field. libavcodec/h264_sei.c | 15 --- libavcodec/h264_sei.h | 3 +-- libavcodec/h264_slice.c | 15 ++- libavcodec/h264dec.c| 5 +++-- 4 files changed, 22 insertions(+), 16 dele

Re: [FFmpeg-devel] [PATCH] h264: Support multi-field closed captions by using AVBufferRef and not resetting per field

2017-07-10 Thread wm4
On Mon, 10 Jul 2017 09:43:09 + Kieran Kunhya wrote: > On Mon, 10 Jul 2017 at 10:39 wm4 wrote: > > > > > > h->frame_recovered = h1->frame_recovered; > > > +if (h1->sei.a53_caption.buf_ref) { > > > +h->sei.a53_caption.buf_ref = > > av_buffer_ref(h1->sei.a53_caption.

Re: [FFmpeg-devel] [PATCH] h264: Support multi-field closed captions by using AVBufferRef and not resetting per field

2017-07-10 Thread Kieran Kunhya
On Mon, 10 Jul 2017 at 10:39 wm4 wrote: > > > h->frame_recovered = h1->frame_recovered; > > +if (h1->sei.a53_caption.buf_ref) { > > +h->sei.a53_caption.buf_ref = > av_buffer_ref(h1->sei.a53_caption.buf_ref); > > +av_buffer_unref(&h1->sei.a53_caption.buf_ref); > > +

Re: [FFmpeg-devel] [PATCH] h264: Support multi-field closed captions by using AVBufferRef and not resetting per field

2017-07-10 Thread wm4
On Sun, 09 Jul 2017 22:59:41 + Kieran Kunhya wrote: > From e8768677511ae5ae9c62c7182a73993e5132f5b8 Mon Sep 17 00:00:00 2001 > From: Kieran Kunhya > Date: Sun, 9 Jul 2017 23:56:14 +0100 > Subject: [PATCH] h264: Support multi-field closed captions by using > AVBufferRef and not resetting per

[FFmpeg-devel] [PATCH] h264: Support multi-field closed captions by using AVBufferRef and not resetting per field

2017-07-09 Thread Kieran Kunhya
$subj 0001-h264-Support-multi-field-closed-captions-by-using-AV.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel