<div class="gmail_quote">On Wed, Apr 7, 2010 at 10:07 AM, brezhoneg1 <span dir="ltr"><<a href="mailto: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;">
<table border="0" cellpadding="0" cellspacing="0"><tbody><tr><td style="font-family: inherit; font-style: inherit; font-variant: inherit; font-weight: inherit; font-size: inherit; line-height: inherit; font-size-adjust: inherit; font-stretch: inherit;" valign="top">
<br>hi,<br><br>   I fear things are a lot more complicated<br><br>    -  There is not one popmenu menu but four independent popmenus (PopMenu, VideoPopupMenu, AudioPopupMenu, MiscPopupMenu)<br>    (see dialogs_provider.cpp around line 162)<br>
</td></tr></tbody></table></blockquote><div><br>I think other ones besides PopupMenu are not used anywhere.<br><br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<table border="0" cellpadding="0" cellspacing="0"><tbody><tr><td style="font-family: inherit; font-style: inherit; font-variant: inherit; font-weight: inherit; font-size: inherit; line-height: inherit; font-size-adjust: inherit; font-stretch: inherit;" valign="top">
<br>    - today's implementation processes PopMenu in a certain way (with p_intf->p_sys->p_popup_menu ) and the three others in another way.<br></td></tr></tbody></table></blockquote><div><br>The three others are just plain leaks and wrong, so if they are ever going to be used, they should do the same as PopupMenu does.<br>
 </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><table border="0" cellpadding="0" cellspacing="0"><tbody><tr><td style="font-family: inherit; font-style: inherit; font-variant: inherit; font-weight: inherit; font-size: inherit; line-height: inherit; font-size-adjust: inherit; font-stretch: inherit;" valign="top">
<br> To sum it up<br><br>    - there is much work to do for menus<br></td></tr></tbody></table></blockquote><div><br>Yes, I agree, I guess everyting could be more elegant, without even needing MenuItemData.<br><br> </div>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><table border="0" cellpadding="0" cellspacing="0"><tbody><tr><td style="font-family: inherit; font-style: inherit; font-variant: inherit; font-weight: inherit; font-size: inherit; line-height: inherit; font-size-adjust: inherit; font-stretch: inherit;" valign="top">
    - Since it's feature freeze, my patch is still the shortest way to work out<br>       the problem, yet not redesign the whole menu thing<br>    - This patch is necessary because we cannot afford to leave vlc objects unreleased</td>
</tr></tbody></table></blockquote><div><br>Please try my attached patch (locally revert your commit [8d8bd80666aff] before, or my patch won't apply).<br>How short can it be? :)<br><br>Best regards,<br>Jakob<br> </div>
</div><br>