<table cellspacing="0" cellpadding="0" border="0" ><tr><td valign="top" style="font: inherit;"><br><br>--- En date de : <b>Mar 6.4.10, Jakob Leben <i><jakob.leben@gmail.com></i></b> a écrit :<br><blockquote style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px;"><br>De: Jakob Leben <jakob.leben@gmail.com><br>Objet: Re: [vlc-devel] [vlc-commits] commit: qt4: Don't create QMenu without parents (Erwan Tulou )<br>À: "Mailing list for VLC media player developers" <vlc-devel@videolan.org><br>Date: Mardi 6 avril 2010, 17h28<br><br><div id="yiv225035630"><div class="gmail_quote">On Tue, Apr 6, 2010 at 5:25 PM, Jakob Leben <span dir="ltr"><<a rel="nofollow" ymailto="mailto:jakob.leben@gmail.com" target="_blank" href="/mc/compose?to=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 rel="nofollow" ymailto="mailto:brezhoneg1@yahoo.fr" target="_blank" href="/mc/compose?to=brezhoneg1@yahoo.fr">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 rel="nofollow" ymailto="mailto:jb@videolan.org" target="_blank" href="/mc/compose?to=jb@videolan.org">jb@videolan.org</a>> a écrit :<br>
<br>
> De: Jean-Baptiste Kempf <<a rel="nofollow" ymailto="mailto:jb@videolan.org" target="_blank" href="/mc/compose?to=jb@videolan.org">jb@videolan.org</a>><br>
> Objet: Re: [vlc-devel] [vlc-commits] commit: qt4: Don't create QMenu without parents (Erwan Tulou )<br>
> À: <a rel="nofollow" ymailto="mailto:vlc-devel@videolan.org" target="_blank" href="/mc/compose?to=vlc-devel@videolan.org">vlc-devel@videolan.org</a><br>
> Cc: <a rel="nofollow" ymailto="mailto:vlc-commits@videolan.org" target="_blank" href="/mc/compose?to=vlc-commits@videolan.org">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 rel="nofollow" ymailto="mailto:git@videolan.org" target="_blank" href="/mc/compose?to=git@videolan.org">git@videolan.org</a><br>
> wrote :<br>
> > vlc | branch: master | Erwan Tulou <<a rel="nofollow" ymailto="mailto:erwan10@videolan.org" target="_blank" href="/mc/compose?to=erwan10@videolan.org">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>
</div><br><br><br>You're right about this p_intf->p_sys->p_popup_menu being buggy and needing some fixes (this is an independent matter anyway)<br><br>Erwan10<br></blockquote></td></tr></table><br>