[FFmpeg-cvslog] r12241 - trunk/libavformat/mov.c

[Baptiste Coudurier]

Reimar D?ffinger wrote:
> On Wed, Feb 27, 2008 at 10:27:43AM +0100, Baptiste Coudurier wrote:
>> Reimar D?ffinger wrote:
>>>> When you're done whining, perhaps you could help think of a proper
>>>> solution to handle situations like this.
>>> Well, the only solution I consider proper is an external tool that
>>> transfers the index to the "external" file so it becomes playable.
>>> But since I guess this is not enough for you, I think the only real
>>> option I can think of is an optional callback function to open a
>>> given resource.
>>> The question then would be the API. My idea would be something like:
>>> open_cb(AVFormatContext *ctx, uint8_t *id, int id_len);
>>> A "void *" of user data would not be bad either, though that probably
>>> goes into AVFormatContext as a new member.
>>> Possibly also a callback to close, though at least for this
>>> functionality this is not necessary.
>>>
>> And what about the reading through ByteIOContext ?
> 
> Uh, the function is supposed to be
> ByteIOContext *open_cb(AVFormatContext *ctx, uint8_t *id, int id_len);
>
>> Your solution is messy and much more complicated.
> 
> Yours is not even remotely acceptable to me.

I didn't read any valid reason yet.
Oh, and feel free to improve mplayer demuxer.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312