Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-29 Thread wm4
On Mon, 28 Jul 2014 17:21:34 +0200 Nicolas George wrote: > Le decadi 10 thermidor, an CCXXII, Oliver Fromme a écrit : > > Ah! Thank you very much for pointing me to the concat demuxer. > > I wasn't aware that it can be (ab)used to declare the presence > > of streams in an input file. I will def

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-29 Thread wm4
On Mon, 28 Jul 2014 10:43:13 +0200 Nicolas George wrote: > Le decadi 10 thermidor, an CCXXII, Eli Kara a écrit : > > It would solve the problem for DVDs with structure, which is probably what > > most people have when they rip a DVD. > > Certainly. But the other scenarios where the user only hav

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-29 Thread Michael Niedermayer
On Sun, Jul 27, 2014 at 10:52:26PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Some DVD dumps contain subtitles that start after more than 2G, > attached patch should allow to specify sufficiently large > values for -probesize. > > Please review, Carl Eugen > doc/APIchanges |4 +

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-29 Thread Carl Eugen Hoyos
Oliver Fromme fromme.com> writes: > > http://thread.gmane.org/gmane.comp.video.ffmpeg.devel/104257/focus=140297 > > That thread seems to center on DVD playback with ffplay, > including menu navigation. > My goal is to enable ffmpeg to be able read one title > from a DVD and encode it. I didn'

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-28 Thread Carl Eugen Hoyos
Oliver Fromme fromme.com> writes: > That's why I appreciate the patch very much > that Carl Eugen has created. Did you test the patch? I think it will not get applied without a test. [...] > Maybe I'll try to make a patch that can do this If you consider to invest time in DVD reading, please

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-28 Thread Nicolas George
Le decadi 10 thermidor, an CCXXII, Oliver Fromme a écrit : > Ah! Thank you very much for pointing me to the concat demuxer. > I wasn't aware that it can be (ab)used to declare the presence > of streams in an input file. I will definitely give that a try. Please let me know of any issue. > Well,

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-28 Thread Nicolas George
Le decadi 10 thermidor, an CCXXII, Oliver Fromme a écrit : > There's another problem: The palette of the subtitle streams > is contained in the IFO files, too. So, in order to get the > palette right, my script uses "mencoder -vobsubout", then > greps the palette from the resulting .idx file and

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-28 Thread Nicolas George
Le decadi 10 thermidor, an CCXXII, Eli Kara a écrit : > It would solve the problem for DVDs with structure, which is probably what > most people have when they rip a DVD. Certainly. But the other scenarios where the user only have a big MPEG-PS file and nothing else, are likely too. > Just out of

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-28 Thread Eli Kara
> From: ffmpeg-devel-boun...@ffmpeg.org > [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of Nicolas George > Sent: Monday, July 28, 2014 1:10 AM > That would be nice, thanks for proposing. But that would not fix the issue: > not all MPEG-PS files come with a DVD structure, even ones with subt

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-27 Thread Nicolas George
Le decadi 10 thermidor, an CCXXII, wm4 a écrit : > I have the impression that there are big hacks going on to compensate > for that, such as trying to read 2GB without doing anything just to > find all subtitle streams. The IFO files of a DVD structure are already a big hack, because MPEG-PS is a

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-27 Thread wm4
On Sun, 27 Jul 2014 23:56:21 +0200 Nicolas George wrote: > Le nonidi 9 thermidor, an CCXXII, wm4 a écrit : > > DVD images have metadata that tell you the number of subtitles in > > advance. > > Surely you are aware that ffmpeg is not capable to read it? I have the impression that there are big

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-27 Thread Nicolas George
Le nonidi 9 thermidor, an CCXXII, wm4 a écrit : > DVD images have metadata that tell you the number of subtitles in > advance. Surely you are aware that ffmpeg is not capable to read it? Regards, -- Nicolas George signature.asc Description: Digital signature

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-27 Thread wm4
On Sun, 27 Jul 2014 22:52:26 +0200 Carl Eugen Hoyos wrote: > Hi! > > Some DVD dumps contain subtitles that start after more than 2G, > attached patch should allow to specify sufficiently large > values for -probesize. > > Please review, Carl Eugen DVD images have metadata that tell you the n

[FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-27 Thread Carl Eugen Hoyos
Hi! Some DVD dumps contain subtitles that start after more than 2G, attached patch should allow to specify sufficiently large values for -probesize. Please review, Carl Eugen diff --git a/doc/APIchanges b/doc/APIchanges index c4e7790..e114df4 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@