[MPlayer-cvslog] CVS: main/libmpdemux aviheader.c,1.63,1.64

[Michael Niedermayer]

Hi

On Saturday 01 January 2005 14:17, Reimar Döffinger wrote:
[...]
> > > 3) That it has to use callbacks for comparison puts limits on
> > > performance, although the implementation (usually?) is much better I
> > > have to admit.
> >
> > well the callback isnt needed, its just a matter of the compiler being
> > able to inline it, which is easily possible for these simple cases
>
> I think the qsort function is already compiled (in the libc) so how
> should the compiler be able to do inlining?

well there could be a static inline qsort() or a qsort macro, dunno if the 
standrad allows that though


>
> Anyway, what do you think about the attached patch? Would that be okay?

iam not the maintainer of libmpdemux/avi*, but it looks ok

[...]
-- 
Michael

"I do not agree with what you have to say, but I'll defend to the death your
right to say it." -- Voltaire