<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2013/9/11 Luca Barbato <span dir="ltr"><<a href="mailto:lu_zero@gentoo.org" target="_blank">lu_zero@gentoo.org</a>></span><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class="im">On 11/09/13 21:01, David Fuhrmann wrote:<br>
> Hi,<br>
><br>
> I just retested the latest patch which suppose to fix the VDA<br>
> memleaks<br>
> (<a href="http://lists.libav.org/pipermail/libav-devel/2013-July/048949.html" target="_blank">http://lists.libav.org/pipermail/libav-devel/2013-July/048949.html</a>)<br>
> and it seems that it works fine. As discussed with lu_zero, this<br>
> patch might be also applied to the libav repository, with some minor<br>
> changes. I am not sure, what the status is about that, Luca?<br>
<br>
</div>It is currently pending since I wanted some consensus in dropping the<br>
namespace breakage and move the initialization code in vlc.<br>
<br>
The fix would be folded inside this change.<br>
<div class="im"><br>
> Nevertheless, if it does not get applied for libav soon, I want to<br>
> suggest to apply the patch locally inside the VLC contrib, to have<br>
> the fix inside the VLC 2.1 release and to avoid deferring the VLC<br>
> release even more because of that. Please see the two patches for<br>
> that attached to this email.<br>
<br>
</div>I'll merge it if you are in hurry for vlc 2.1 the patch itself is fine IMHO.<br></blockquote><div><br></div><div style>Hi Luca,</div><div style>It would be cool if you can do that. IMO it is better to merge the memleak patch separately in order to have it in time for 2.1 and to keep the code changes to a minimum before the release.</div>
<div style><br></div><div style>With best regards,</div><div style>David</div></div></div></div>