Re: [FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-22 Thread Michael Niedermayer
On Fri, Sep 19, 2014 at 07:15:09PM +0200, wm4 wrote: > On Fri, 19 Sep 2014 07:33:26 -0700 > Nicholas Robbins wrote: > > > Ok, not to beat a dead horse, but I think I've gotten this working. I think > > it might work better in lavc/dvdsubdec.c. AFAICT this works and doesn't > > produce mangled t

Re: [FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-19 Thread Michael Niedermayer
On Fri, Sep 19, 2014 at 10:57:41AM -0700, Nicholas Robbins wrote: > > On Friday, September 19, 2014 12:31 PM, Carl Eugen Hoyos > > wrote: > > > > > > > Nicholas Robbins ffmpeg.org> writes: > > > >> +{ "only_dec_forced", "Only decode forced > > subtitles", > > > > Please use the same

Re: [FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-19 Thread Nicholas Robbins
> On Friday, September 19, 2014 1:15 PM, wm4 wrote: > > Sorry, but didn't the discussion go towards that ffmpeg.c should filter > out subtitles flagged as forced? > > And in fact, the PGS decoder should be changed to flag forced subs > correctly. I've never peeked inside ffmpeg.c so I don't

Re: [FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-19 Thread Nicholas Robbins
> On Friday, September 19, 2014 12:31 PM, Carl Eugen Hoyos > wrote: > > Nicholas Robbins ffmpeg.org> writes: > >> +{ "only_dec_forced", "Only decode forced > subtitles", > > Please use the same option name as for pgs subtitles. > > Carl Eugen Ok, I'll do that. Also I noticed that

Re: [FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-19 Thread wm4
On Fri, 19 Sep 2014 07:33:26 -0700 Nicholas Robbins wrote: > Ok, not to beat a dead horse, but I think I've gotten this working. I think > it might work better in lavc/dvdsubdec.c. AFAICT this works and doesn't > produce mangled tracks. Patch at end of email. (Sorry for the inline patch) > If

Re: [FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-19 Thread Michael Niedermayer
On Fri, Sep 19, 2014 at 07:33:26AM -0700, Nicholas Robbins wrote: > Ok, not to beat a dead horse, but I think I've gotten this working. I think > it might work better in lavc/dvdsubdec.c. AFAICT this works and doesn't > produce mangled tracks. Patch at end of email. (Sorry for the inline patch)

Re: [FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-19 Thread Carl Eugen Hoyos
Nicholas Robbins ffmpeg.org> writes: > +{ "only_dec_forced", "Only decode forced subtitles", Please use the same option name as for pgs subtitles. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo

[FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-19 Thread Nicholas Robbins
Ok, not to beat a dead horse, but I think I've gotten this working. I think it might work better in lavc/dvdsubdec.c. AFAICT this works and doesn't produce mangled tracks. Patch at end of email. (Sorry for the inline patch) If this is a good patch, I'll add the appropriate documentation and do a