[MPlayer-dev-eng] Re: [PATCH] Save memory in demux_rtp.cpp

Nico Sabbi nicola_sabbi at fastwebnet.it
Mon Feb 26 23:46:41 CET 2007


Reimar Döffinger wrote:
> Hello,
> On Mon, Feb 26, 2007 at 11:06:19PM +0100, Nico Sabbi wrote:
> [...]
> 
>>new_demux_packet() and resize_demux_packet() both store the length of 
>>the buffer in dp->len, so at any time dp->len is the amount of memory
>>allocated;
>>resize_demux_packet(dp, dp->len); is equivalent to a no-op, as you can 
>>see:
> 
> 
> Unless somewhere dp->len is changed without calling
> resize_demux_packet (possibly out of performance considerations)...
> 

I feared the same, but in this case Carl should point it out and 
remove that code. Tampering with demux_packet... have we got to this 
point?

-- 
"Without a frontend, mplayer is useless" - someone in mplayer-users




More information about the MPlayer-dev-eng mailing list