<div class="gmail_quote">On Sat, Apr 3, 2010 at 1:26 PM, 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;">
<div><div></div><div class="h5"><br>
><br>
> >Qt: delete unneeded MenuItemData (Jakob Leben<br>
> >eacddb63e8a5bda42395ee79b5b27db6408] | committer: Jakob Leben<br>
> ><br>
> >Qt: delete unneeded MenuItemData<br>
> ><br></div></div></blockquote><div> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div><div class="h5">
> This patch was singled out as the one causing a regression<br>
> for skins2 (a user of qt4 menus),  namely skins2 yet again<br>
> crashing at exit on both win32 and Linux.<br>
><br>
> When using qFindChild, are we sure there is a unique direct<br>
> child for an action ? Technically, there could be multiple<br>
> ones, in which case the one being deleted<br>
> is may still be addressed elsewhere.  (qt4 doc seems to say<br>
> it is undefined in this case)<br>
><br>
> As a matter of fact, just reverting this patch does solve the problem.<br>
><br>
> Was this patch correcting a specific issue or just intended<br>
> as a cosmetic ?<br>
><br>
> Regards<br>
> Erwan10<br>
<br>
</div></div>Edit: the qt4 interface has got the very same problem<br>
  - start vlc -I qt<br>
  - exit via the popupmenu (rightclick on mouse)<br>
vlc then crashes.<br></blockquote><div><br>Hi, I will look at this problem this evening. Thanks for reporting!<br><br>Jakob <br></div></div>