[MPlayer-dev-eng] [PATCH] semi-fixing mencoder lavc on big endian

Colin Leroy colin at colino.net
Tue Jul 30 21:35:06 CEST 2002


On Tue, 30 Jul 2002 20:29:27 +0200 Arpi <arpi at thot.banki.hu> wrote:

Hi Arpi,
> > -     
> > write_avi_chunk(f,ckidAVINEWINDEX,16*muxer->idx_pos,muxer->idx);
> > +{
> > +      int pos=16*muxer->idx_pos;
> > +      le2me_AVIINDEXENTRY(muxer->idx);
> > +      write_avi_chunk(f,ckidAVINEWINDEX,pos,muxer->idx); /*
> > AVIINDEXENTRY */
> > +      le2me_AVIINDEXENTRY(muxer->idx);
> > +}
>
> this is bad...
> 16*muxer->idx_pos is not a 'pos', but the size of the whole index chunk,
> so 16*number_of_index_entries.

Ok, sorry for this;

> so, le2me_AVIINDEXENTRY() shoudl be done for every index entries (in a
> loop), not only for the first one.

I don't find anything to do it nicely, do you have an idea for me ?

Thanks,
-- 
Colin
http://www.colino.net/



More information about the MPlayer-dev-eng mailing list