[MPlayer-dev-eng] [PATCH] 1 memcpy less in lavf demuxer
Michael Niedermayer
michaelni at gmx.at
Sat Jul 10 04:24:58 CEST 2010
On Tue, Jun 29, 2010 at 04:41:10PM +0200, Reimar Döffinger wrote:
> On Tue, Jun 29, 2010 at 02:46:35PM +0200, Michael Niedermayer wrote:
> > On Sat, Jun 26, 2010 at 08:14:27PM +0200, Reimar Döffinger wrote:
> > > On Tue, Jun 22, 2010 at 11:36:39PM +0200, Michael Niedermayer wrote:
> > > > + if(pkt.destruct == av_destruct_packet){
> > > > dp=malloc(sizeof(demux_packet_t));
> > > > dp->len=pkt.size;
> > > > dp->next=NULL;
> > > > dp->refcount=1;
> > > > dp->master=NULL;
> > > > dp->buffer=pkt.data;
> > >
> > > Um, won't MPlayer use free() instead of av_free() on this,
> > > thus breaking on Windows?
> > > I know we replaced some malloc/free with the av_ functions,
> > > but not the demuxer ones I think...
> >
> > hmm, then what about:
>
> Should be ok (if you don't mind the hackishness of it), though
> generally using av_malloc/av_free for the demuxer packets may
> be a better long-term solution...
applied
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100710/6ce53722/attachment.pgp>
More information about the MPlayer-dev-eng
mailing list