[vlc-devel] [PATCH] Fix-for-TS-Muxer-memory-leak (Part II)

Rafaël Carré rafael.carre at gmail.com
Sun Jan 24 16:46:51 CET 2010


On Sat, 23 Jan 2010 21:51:40 -0500
Robert Krakora <rob.krakora at messagenetsystems.com> wrote:

> The patch looks good.  I have had it running on over 100 Linux-based
> mediaports since I came up with it in September and have seen no
> reoccurance of the memory leak.

I can't review the patch but here some remarks made on irc:

19:37 < nefrir> mmh the code style is wrong (ie not consistant)
19:37 < funman> also the for (i=-1...) looks weird
19:39 < nefrir> and the patch protects only against audio missing I
think
19:39 < nefrir> (his use case)
19:40 < j-b> beurk at the patch then

JPsaman could you have a look at it ?


-- 
Rafaël Carré
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20100124/0124c003/attachment.sig>


More information about the vlc-devel mailing list