Re: [FFmpeg-devel] [PATCH 4/4] avcodec/decode: Pass on the Closed Captions Side Data

2017-09-01 Thread Jeyapal, Karthick
>Actually in ff_init_buffer_info is a mapping of packet sidedata to >frame sidedata, if you just add the new CC sidedata type there, the >existing code should already handle it, from what i can tell. > >- Hendrik Thanks for that guidance. It makes perfect sense to handle it in

Re: [FFmpeg-devel] [PATCH 4/4] avcodec/decode: Pass on the Closed Captions Side Data

2017-08-31 Thread Hendrik Leppkes
On Thu, Aug 31, 2017 at 1:21 PM, Jeyapal, Karthick wrote: >>Maybe this should be handled somewhere in generic code? This is hardly >>a v210 property at all. > > Yes, I agree. It is better to handle it in the generic code. Have attached a > patch, where the same is handled in

Re: [FFmpeg-devel] [PATCH 4/4] avcodec/decode: Pass on the Closed Captions Side Data

2017-08-31 Thread Jeyapal, Karthick
>Maybe this should be handled somewhere in generic code? This is hardly >a v210 property at all. Yes, I agree. It is better to handle it in the generic code. Have attached a patch, where the same is handled in decode.c Thanks and regards, Karthick