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

Jean-Paul Saman jpsaman at gmail.com
Mon Jan 25 15:47:09 CET 2010


On Sun, Jan 24, 2010 at 4:46 PM, Rafaël Carré <rafael.carre at gmail.com> wrote:
> 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 ?
>

Which patch of the 2 is supposed to be the correct one?

gtz
Jean-Paul Saman



More information about the vlc-devel mailing list