Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-07 Thread wm4
On Tue, 7 Jul 2015 12:35:01 + (UTC) Carl Eugen Hoyos ceho...@ag.or.at wrote: Paul B Mahol onemda at gmail.com writes: How can we proceed? Adding side data. I was under the impression that side date is used for communication between different libraries. Is that wrong? Yes.

Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-07 Thread Michael Niedermayer
On Tue, Jul 07, 2015 at 12:49:23PM +, Paul B Mahol wrote: On 7/7/15, Carl Eugen Hoyos ceho...@ag.or.at wrote: Paul B Mahol onemda at gmail.com writes: How can we proceed? Adding side data. I was under the impression that side date is used for communication between different

Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-07 Thread Carl Eugen Hoyos
wm4 nfxjfg at googlemail.com writes: Adding side data. I was under the impression that side date is used for communication between different libraries. Is that wrong? Yes. So how can avcodec_string() use side data? Carl Eugen ___

Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-07 Thread Michael Niedermayer
On Tue, Jul 07, 2015 at 10:12:05PM +, Carl Eugen Hoyos wrote: Michael Niedermayer michaelni at gmx.at writes: maybe adding the field to AVCodecInternal would work and be a solution ? enc-internal is NULL in avcodec_string(). (That was my first try.) then i think your original

Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-07 Thread Carl Eugen Hoyos
Michael Niedermayer michaelni at gmx.at writes: maybe adding the field to AVCodecInternal would work and be a solution ? enc-internal is NULL in avcodec_string(). (That was my first try.) But I also wonder why library users would prefer to jump through loops to know if a video stream

Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-07 Thread Carl Eugen Hoyos
Paul B Mahol onemda at gmail.com writes: How can we proceed? Adding side data. I was under the impression that side date is used for communication between different libraries. Is that wrong? This is about storing information about video streams and using this information within

Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-07 Thread Paul B Mahol
On 7/7/15, Carl Eugen Hoyos ceho...@ag.or.at wrote: Paul B Mahol onemda at gmail.com writes: How can we proceed? Adding side data. I was under the impression that side date is used for communication between different libraries. Is that wrong? This is about storing information about

Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-07 Thread Carl Eugen Hoyos
Carl Eugen Hoyos cehoyos at ag.or.at writes: As in attached? If anything it should use the automatic accessor macros, I don't think that works, no setter is needed. but I still find it a silly solution. Then please suggest another solution. Ping. How can we proceed? Carl

Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-06 Thread Carl Eugen Hoyos
Anshul anshul.ffmpeg at gmail.com writes: We had long disscussion last year for same thing, and i ended up in libavfilter movie, Do you mean a discussion about telling users about Closed Captions in video streams or a discussion about patches to implement decoding of Closed Captions? I am

Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-06 Thread Anshul
We had long disscussion last year for same thing, and i ended up in libavfilter movie, though i dont have any problem with this patch. We also used side data for same thing if u want u can look in that directions. my opinion would be to treat h264 caption in same way as it is treated in mpeg,

Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-06 Thread Carl Eugen Hoyos
On Monday 06 July 2015 06:18:14 pm Carl Eugen Hoyos wrote: Hendrik Leppkes h.leppkes at gmail.com writes: Attached patch adds Closed Captions to the codec dump if the video stream contains them. Adding fields to a public context which should never be accessed by the public is a big

Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-06 Thread Carl Eugen Hoyos
Hendrik Leppkes h.leppkes at gmail.com writes: Attached patch adds Closed Captions to the codec dump if the video stream contains them. Adding fields to a public context which should never be accessed by the public is a big no-no. What is the alternative? Carl Eugen

Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-06 Thread Hendrik Leppkes
On Mon, Jul 6, 2015 at 6:32 PM, Carl Eugen Hoyos ceho...@ag.or.at wrote: On Monday 06 July 2015 06:18:14 pm Carl Eugen Hoyos wrote: Hendrik Leppkes h.leppkes at gmail.com writes: Attached patch adds Closed Captions to the codec dump if the video stream contains them. Adding fields to a

Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-06 Thread Carl Eugen Hoyos
Hendrik Leppkes h.leppkes at gmail.com writes: On Mon, Jul 6, 2015 at 6:32 PM, Carl Eugen Hoyos wrote: On Monday 06 July 2015 06:18:14 pm Carl Eugen Hoyos wrote: Hendrik Leppkes h.leppkes at gmail.com writes: Attached patch adds Closed Captions to the codec dump if the video stream

[FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-06 Thread Carl Eugen Hoyos
Hi! Attached patch adds Closed Captions to the codec dump if the video stream contains them. Please comment, Carl Eugen diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index e367399..c39a241 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@ -3171,6 +3171,14 @@ typedef

Re: [FFmpeg-devel] [PATCH]Tell users about Closed Captions in video streams

2015-07-06 Thread Hendrik Leppkes
On Mon, Jul 6, 2015 at 5:53 PM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Hi! Attached patch adds Closed Captions to the codec dump if the video stream contains them. Adding fields to a public context which should never be accessed by the public is a big no-no. - Hendrik