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

Michael Niedermayer michaelni at gmx.at
Sat Jan 1 14:52:26 CET 2005


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


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

More information about the MPlayer-cvslog mailing list