<p>On Aug 1, 2011 10:25 AM, "Alexandre Julliard" <<a href="mailto:julliard@winehq.org">julliard@winehq.org</a>> wrote:<br>
><br>
> Andy Clayton <<a href="mailto:q3aiml@gmail.com">q3aiml@gmail.com</a>> writes:<br>
><br>
> > + itemMenu = MENU_GetMenu(hMenu);<br>
> > + if (itemMenu == NULL)<br>
> > + return FALSE;<br>
> > +<br>
> > + if ((itemMenu->Height == 0 || itemMenu->Width == 0) && itemMenu->wFlags ^ MF_POPUP)<br>
><br>
> That flag test doesn't make sense. What are you trying to check here?<br>
></p>
<p>I am trying to check is if it is a menu bar, because in that case GetMenuItemRect works so long as the menu is attached to a window (according to MSDN, anyways). Is there a better way I should be checking this? I guess I overlooked MF_SYSMENU. I also now see that I should instead be calling MenuBarCalcSize, and that my testing has found it returning values even if not attached via SetMenu, though maybe they are not meaningful. Clearly I need to take a second look at this.</p>
<p>Sorry for so many mistakes.</p>
<p>Andy</p>