[vlc-devel] [PATCH 1/4] libmp4: Parse StraBox and SmooBox

[Frederic YHUEL]

On Wed, Jun 27, 2012 at 3:07 PM, Rémi Denis-Courmont <remi at remlab.net> wrote:
> Le mercredi 27 juin 2012 15:46:33 Frederic YHUEL, vous avez écrit :
>> If avformat is used, we would get the error: "moov atom not found". It
>> is the end of the world?
>>
>> avformat is used only if Open() returns VLC_EGENERIC, in VLC's MP4
>> demux, as far as know. Or if we force it.
>>
>> If I write code to build a genuine moov box, by the time I finish,
>> Smooth Streaming would be completely dead :-)
>
> I'm not saying you should synthetize a moov box. I am saying you should not
> synthetize anything.
>
>> Besides, that design has been approved by j-b two months ago. I'm not
>> saying he is stronger or anything, but I don't have time to change
>> everything every quarter :-s
>
> Actually Laurent knows best how the MP4 demux works. But in my understanding,
> private communication between plugins is done with private control requests,
> NOT with in-band data. I already told you that months ago.
>

Indeed, and that was how I did first, but then Laurent rejected my
solution. That led to a discussion whose conclusion was that a
access_demux would fit better.

Then I asked if I could synthesize an init fragment in the
stream_filter module, and j-b told me on IRC that was not completely
stupid.

Eventually we will have to port all this modules, HLS, DASH ans Smooth
Streaming, into a access_module, as you said. It is really needed for
HLS which doesn't support resolution switches.

And I think it's possible to write a unified module for DASH and Smooh
Streaming.

But in he meantime, I think what I did is not that ugly. Actually,
these MP4 patches bring support for resolution switches to both DASH
and Smooth Streaming. And the latest version of the Smooh Streaming
module I'm going to submit works pretty well.

Best regards
Frédéric