[Ffmpeg-devel] [PATCH] cook compatible decoder

Michael Niedermayer michaelni
Fri Nov 4 15:00:56 CET 2005


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:
> >  
> >
> >>http://tranquillity.campus.luth.se/~banan/cook/
> >>
> >>patch -p0 <cookffmpeg.patch
> >>
> >>Please review, test and comment. The rm demuxer patch is made by Roberto
> >>Togni.
> >>I think the decoder wont work on BE systems.
> >>    
> >>
> >
> >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

[...]

-- 
Michael





More information about the ffmpeg-devel mailing list