[MPlayer-dev-eng] Re: native vs libavformat demuxers

[Michael Niedermayer]

Hi

On Mon, Feb 26, 2007 at 12:41:38AM +0100, Nico Sabbi wrote:
> Diego Biurrun wrote:
> >On Sun, Feb 25, 2007 at 11:25:13PM +0100, Nico Sabbi wrote:
> >
> >>Since you all want to remove it, at least let's begin with removing 
> >>code that is obsolete
> >>(e.g. demux_aac, demux_audio and demux_avi come to mind, maybe more)
> >
> >
> >Hehe, I was thinking more of the obscure stuff like demux_film and
> >whatnot.
> >
> >I have a feeling that most of the libavformat demuxers except for
> >RealMedia are probably better than our native ones.  Did anybody compare
> >them in detail and/or run benchmarks?
> >
> >Diego
> 
> never done, but there's one main problem to fix in the interface 
> between libmpdemux and demux_lavf: removing the need to
> memcpy the payload from the av_packet to demux_packet;
> maybe extending the av_packet so as to report the amount of memory
> needed and passing to lavf demux_packet's buffer?
> or maybe something more good looking

what about the if(0) code in demux_lavf.c ? (yes i know its ugly ...)

anyway, supporting user supplied buffers in AVPacket should be trivial
would a callback like AVFormatContext.new_packet() do? this seems like
the most flexible solution but maybe it has issues?

suggestions very welcome!

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

it is not once nor twice but times without number that the same ideas make
their appearance in the world. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20070226/bf8b94c2/attachment.pgp>