<div class="gmail_quote">On Tue, Apr 6, 2010 at 5:25 PM, Jakob Leben <span dir="ltr"><<a href="mailto:jakob.leben@gmail.com">jakob.leben@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 class="gmail_quote"><div class="im">On Tue, Apr 6, 2010 at 5:19 PM, brezhoneg1 <span dir="ltr"><<a href="mailto:brezhoneg1@yahoo.fr" target="_blank">brezhoneg1@yahoo.fr</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;">

<br>
<br>
--- En date de : Mar 6.4.10, Jean-Baptiste Kempf <<a href="mailto:jb@videolan.org" target="_blank">jb@videolan.org</a>> a écrit :<br>
<br>
> De: Jean-Baptiste Kempf <<a href="mailto:jb@videolan.org" target="_blank">jb@videolan.org</a>><br>
> Objet: Re: [vlc-devel] [vlc-commits] commit: qt4: Don't create QMenu without parents (Erwan Tulou )<br>
> À: <a href="mailto:vlc-devel@videolan.org" target="_blank">vlc-devel@videolan.org</a><br>
> Cc: <a href="mailto:vlc-commits@videolan.org" target="_blank">vlc-commits@videolan.org</a><br>
> Date: Mardi 6 avril 2010, 16h58<br>
<div>> On Tue, Apr 06, 2010 at 04:56:06PM<br>
> +0200, <a href="mailto:git@videolan.org" target="_blank">git@videolan.org</a><br>
> wrote :<br>
> > vlc | branch: master | Erwan Tulou <<a href="mailto:erwan10@videolan.org" target="_blank">erwan10@videolan.org</a>><br>
> | Tue Apr  6 16:40:11 2010 +0200|<br>
> [8d8bd80666aff8073f1ccd37a41cd1786096b0c3] | committer:<br>
> Erwan Tulou<br>
> ><br>
> > qt4: Don't create QMenu without parents<br>
> ><br></div></blockquote></div><div><br>In my opinion, this commit is unnecessary bloat. All that was needed to fix previously mentioned bug was to delete p_intf->p_sys->p_popup_menu when qt interface closes. <br>
</div></div>
</blockquote></div><br>Btw, with this commit, problems might also arise when a popped up menu is deleted because it's parent is, but p_intf->p_sys->p_popup_menu is not set to NULL.<br>