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

[Robert Krakora]

Rafaël

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>
> 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
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
>
>
**no-whitespace-ts.diff e-mailed by Rafaël.

-- 
Rob Krakora
Senior Software Engineer
MessageNet Systems
101 East Carmel Dr. Suite 105
Carmel, IN 46032
(317)566-1677 Ext. 206
(317)663-0808 Fax
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20100125/a9f4ceb2/attachment.html>