Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-10-03 Thread Michael Niedermayer
On Tue, Sep 29, 2015 at 01:19:34PM -0400, DeHackEd wrote: > Assumes 'GA94' format (ATSC standard) > > Signed-off-by: DHE > --- > doc/encoders.texi| 4 > libavcodec/libx264.c | 45 + > 2 files changed, 49 insertions(+) applied thanks [...]

Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-29 Thread DeHackEd
Assumes 'GA94' format (ATSC standard) Signed-off-by: DHE --- doc/encoders.texi| 4 libavcodec/libx264.c | 45 + 2 files changed, 49 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi index 3550bcc..f2d46dc 100644 --- a/doc/enco

Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-27 Thread DeHackEd
Assumes 'GA94' format (ATSC standard) Signed-off-by: DHE --- doc/encoders.texi| 4 libavcodec/libx264.c | 46 ++ 2 files changed, 50 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi index 3550bcc..f2d46dc 100644 --- a/doc/enc

Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-26 Thread Dave Yeo
On 09/26/15 03:27 AM, DeHackEd wrote: I think my mail client wrapped some lines. Sorry, this is my first patch-by-email. For Thunderbird, there is an extension, "Toggle Word Wrap" which gives the option of disabling word wrap under the Options menu. Or just attach. Dave _

Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-26 Thread James Almer
On 9/26/2015 8:41 AM, DeHackEd wrote: > +/** > + * 'GA94' is standard in North America for ATSC, but hard > coding > + * this style may not be the right thing to do -- other > formats > + * do exist. This information is not available

Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-26 Thread Michael Niedermayer
On Sat, Sep 26, 2015 at 07:41:32AM -0400, DeHackEd wrote: > Assumes 'GA94' format (ATSC standard) > > Signed-off-by: DHE > --- > doc/encoders.texi| 5 + > libavcodec/libx264.c | 37 + > 2 files changed, 42 insertions(+) > > diff --git a/doc/encoders.

[FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-26 Thread DeHackEd
Assumes 'GA94' format (ATSC standard) Signed-off-by: DHE --- doc/encoders.texi| 5 + libavcodec/libx264.c | 37 + 2 files changed, 42 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi index 3550bcc..aabbda0 100644 --- a/doc/encoders.te

Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-26 Thread Anshul
On 09/26/2015 04:16 PM, Anshul wrote: On 09/26/2015 03:57 PM, DeHackEd wrote: On 09/26/2015 05:26 AM, Anshul wrote: On 09/26/2015 05:46 AM, DeHackEd wrote: Assumes 'GA94' format (ATSC standard) Signed-off-by: DHE --- doc/encoders.texi| 5 + libavcodec/libx264.c | 37 ++

Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-26 Thread Anshul
On 09/26/2015 03:57 PM, DeHackEd wrote: On 09/26/2015 05:26 AM, Anshul wrote: On 09/26/2015 05:46 AM, DeHackEd wrote: Assumes 'GA94' format (ATSC standard) Signed-off-by: DHE --- doc/encoders.texi| 5 + libavcodec/libx264.c | 37 + 2 file

Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-26 Thread DeHackEd
On 09/26/2015 05:26 AM, Anshul wrote: > > > On 09/26/2015 05:46 AM, DeHackEd wrote: >> Assumes 'GA94' format (ATSC standard) >> >> Signed-off-by: DHE >> --- >> doc/encoders.texi| 5 + >> libavcodec/libx264.c | 37 + >> 2 files changed, 42 insertio

Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-26 Thread Anshul
On 09/26/2015 05:46 AM, DeHackEd wrote: Assumes 'GA94' format (ATSC standard) Signed-off-by: DHE --- doc/encoders.texi| 5 + libavcodec/libx264.c | 37 + 2 files changed, 42 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi inde

Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-25 Thread DeHackEd
Assumes 'GA94' format (ATSC standard) Signed-off-by: DHE --- doc/encoders.texi| 5 + libavcodec/libx264.c | 37 + 2 files changed, 42 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi index 3550bcc..bb16dea 100644 --- a/doc/encoders.te

Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-25 Thread DeHackEd
On 09/25/2015 07:45 PM, Carl Eugen Hoyos wrote: > DeHackEd dehacked.net> writes: > >> + item a53cc >> +Import closed captions (which must be ATSC compatible format) into output. >> +At this time only the mpeg2 decoder provides these. > > I thought the h264 decoder also provides them, no? Indee

Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-25 Thread Carl Eugen Hoyos
DeHackEd dehacked.net> writes: > + item a53cc > +Import closed captions (which must be ATSC compatible format) into output. > +At this time only the mpeg2 decoder provides these. I thought the h264 decoder also provides them, no? Carl Eugen ___ ffmp

[FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-25 Thread DeHackEd
Assumes 'GA94' format (ATSC standard) Signed-off-by: DHE --- doc/encoders.texi| 5 + libavcodec/libx264.c | 37 + 2 files changed, 42 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi index 3550bcc..8e3770b 100644 --- a/doc/encoders.te