[FFmpeg-devel] [SCISYS Possible Spam] Re: [SCISYS Possible Spam] Re: [PATCH v5] avformat/mov: Memory optimization with QuickTime/MP4

[Carl Eugen Hoyos]

Am Mo., 6. Jan. 2020 um 15:37 Uhr schrieb Jörg Beckmann
<Joerg.Beckmann at scisys.com>:
>
> > -----Ursprüngliche Nachricht-----
> > Von: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> Im Auftrag von Carl
> > Eugen Hoyos
> > Gesendet: Montag, 6. Januar 2020 15:24
> > An: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
> > Betreff: [SCISYS Possible Spam] Re: [FFmpeg-devel] [SCISYS Possible Spam]
> > Re: [PATCH v5] avformat/mov: Memory optimization with QuickTime/MP4
> >
> > Am Mo., 16. Dez. 2019 um 12:13 Uhr schrieb Jörg Beckmann
> > <Joerg.Beckmann at scisys.com>:
> > >
> > >
> > > > -----Ursprüngliche Nachricht-----
> > > > Von: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> Im Auftrag von
> > > > Carl Eugen Hoyos
> > > > Gesendet: Montag, 16. Dezember 2019 11:50
> > > > An: FFmpeg development discussions and patches
> > > > <ffmpeg-devel at ffmpeg.org>
> > > > Betreff: [SCISYS Possible Spam] Re: [FFmpeg-devel] [PATCH v5]
> > avformat/mov:
> > > > Memory optimization with QuickTime/MP4
> > > >
> > > > Am Mo., 9. Dez. 2019 um 16:05 Uhr schrieb Jörg Beckmann
> > > > <Joerg.Beckmann at scisys.com>:
> > > > >
> > > > > Invents a new option "discard_fragments" for the MP4/Quicktime/MOV
> > decoder.
> > > > >
> > > > > If the option is not set, nothing changes at all. If it is set,
> > > > > old fragments are discarded as far as possible on each call to switch_root.
> > > >
> > > > > For pure audio streams, the memory usage is now constant.
> > > >
> > > > Is it possible to detect this case?
> > > > If possible, the new option could have an "auto" setting.
> > >
> > > I'm not sure whether it really works with all possible stream types.
> >
> > Why is it not possible to detect that the stream only contains audio?
>
> Oh sorry, I misunderstood your question. Of course it is possible to detect
> that it contains only audio. I thought you suggested to detect that is fragmented.
> I'll take a look into the sources to find an appropriate location for the check and
> send the modified patch again. But it will take a few days.
>
> Do you think, "auto" should be the default value?

If your patch means that receiving audio streams allocates less
memory and the audio-only case can be detected, yes.

Carl Eugen