[vlc-devel] [PATCH] Fix-for-TS-Muxer-memory-leak (Part II)
rob.krakora at messagenetsystems.com
Mon Jan 25 16:02:09 CET 2010
On Mon, Jan 25, 2010 at 9:47 AM, Jean-Paul Saman <jpsaman at gmail.com> wrote:
> On Sun, Jan 24, 2010 at 4:46 PM, Rafaël Carré <rafael.carre at gmail.com>
> > 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?
> Jean-Paul Saman
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
**no-whitespace-ts.diff e-mailed by Rafaël.
Senior Software Engineer
101 East Carmel Dr. Suite 105
Carmel, IN 46032
(317)566-1677 Ext. 206
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the vlc-devel