<table cellspacing="0" cellpadding="0" border="0" ><tr><td valign="top" style="font: inherit;">Hi,<br><br>   Please find attached a patch that  could be a tradeoff  (since it's feature frozen time ) ....<br><br>     - removes p_sys->p_popup_menu<br>     - add a static local variable in each of the four Popupmenu to guarantee a singleton in terms of QMenu.<br><br>     Comments welcome<br><br>Regards<br>Erwan10<br><br>--- En date de : <b>Mer 7.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: Mercredi 7
 avril 2010, 16h15<br><br><div id="yiv1897478127"><div class="gmail_quote">On Wed, Apr 7, 2010 at 2:36 PM, Francois Cartegnie <span dir="ltr"><<a rel="nofollow" ymailto="mailto:fcvlcdev@free.fr" target="_blank" href="/mc/compose?to=fcvlcdev@free.fr">fcvlcdev@free.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;">

Storing ui classes references into the library isn't acceptable. </blockquote><blockquote style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;" class="gmail_quote"><div> </div>
</blockquote><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
- define a QVLCMenu interface<br>
- QVLCMenu inherits singleton<br>
- Each popup menu class inherits QVLCMenu (insuring only 1 instance)<br>
- New instance deletes old menu instance (with care of slicing), or recycle<br></blockquote><div><br>To simply implement "singleton" popup menus we don't really need that the whole QVLCMenu inherits singleton. (Note that there is no need to actually get a reference to an instance
of a popup menu. It suffices to call a function that pops the menu up.)<br><br>The p_popup_menu right now is effectively a singleton design. In the light of your remarks, we just move p_popup_menu to QVLCMenu, as it's static member.<br>
<br></div></div>
</div><br>-----La pièce jointe associée suit-----<br><br><div class="plainMail">_______________________________________________<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></div></blockquote></td></tr></table><br>