<font color="#888888">Rafaël</font><br><br><div class="gmail_quote">On Mon, Jan 25, 2010 at 9:47 AM, Jean-Paul Saman <span dir="ltr"><<a href="mailto:jpsaman@gmail.com">jpsaman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div></div><div class="h5">On Sun, Jan 24, 2010 at 4:46 PM, Rafaël Carré <<a href="mailto:rafael.carre@gmail.com">rafael.carre@gmail.com</a>> wrote:<br>
> On Sat, 23 Jan 2010 21:51:40 -0500<br>
> Robert Krakora <<a href="mailto:rob.krakora@messagenetsystems.com">rob.krakora@messagenetsystems.com</a>> wrote:<br>
><br>
>> The patch looks good.  I have had it running on over 100 Linux-based<br>
>> mediaports since I came up with it in September and have seen no<br>
>> reoccurance of the memory leak.<br>
><br>
> I can't review the patch but here some remarks made on irc:<br>
><br>
> 19:37 < nefrir> mmh the code style is wrong (ie not consistant)<br>
> 19:37 < funman> also the for (i=-1...) looks weird<br>
> 19:39 < nefrir> and the patch protects only against audio missing I<br>
> think<br>
> 19:39 < nefrir> (his use case)<br>
> 19:40 < j-b> beurk at the patch then<br>
><br>
> JPsaman could you have a look at it ?<br>
><br>
<br>
</div></div>Which patch of the 2 is supposed to be the correct one?<br>
<br>
gtz<br>
<font color="#888888">Jean-Paul Saman<br>
</font><div><div></div><div class="h5">_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
<br>
</div></div></blockquote></div><br><table class="cf hr" cellpadding="0"><tbody><tr></tr><tr><td><b></b></td></tr></tbody></table>no-whitespace-ts.diff e-mailed by <font style="color: rgb(0, 0, 0);" color="#888888">Rafaël.</font><br clear="all">
<br>-- <br>Rob Krakora<br>Senior Software Engineer<br>MessageNet Systems<br>101 East Carmel Dr. Suite 105<br>Carmel, IN 46032<br>(317)566-1677 Ext. 206<br>(317)663-0808 Fax<br>