[Ffmpeg-devel] [PATCH] cook compatible decoder

[Benjamin Larsson]

Hi,

Michael Niedermayer wrote:

>Hi
>
>On Fri, Nov 04, 2005 at 02:10:16PM +0100, Benjamin Larsson wrote:
>  
>
>>Michael Niedermayer wrote:
>>
>>    
>>
>>>Hi
>>>
>>>On Thu, Nov 03, 2005 at 08:08:09PM +0100, Benjamin Larsson wrote:
>>> 
>>>
>>>      
>>>
>>> [...]
>>>
>>>how should it with that?:
>>>
>>>typedef struct __attribute__((__packed__)){
>>>   uint16_t sub_packet_size;           //8bit large elements
>>>   uint16_t sub_packet_h;
>>>...
>>>COOKextradata *e = avctx->extradata;
>>>   int w=avctx->block_align;
>>>   int h = e->sub_packet_h;
>>>...
>>>
>>>
>>>[...]
>>> 
>>>
>>>      
>>>
>>This is variables handled by the rm demuxer, so I assume that they
>>already are in their native endian. If that was what you hinted at.
>>    
>>
>
>no, thats what i was afraid of, its unacceptable, to do this in the
>demuxer, as cook in anything but rm would be impossble without cook
>specific code in every (de)muxer
>
>[...]
>
>  
>
ok, but wouldn't that need a rewrite of the rm demuxer ? Currently the
the rm container and codecs are tightly integrated. So I'm not entirely
sure how to make that possible.

MvH
Benjamin Larsson