[vlc-devel] [PATCH] Fix-for-TS-Muxer-memory-leak (Part II)
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
> 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?
More information about the vlc-devel