Recent change causes bad QuickBooks behaviour(?)

[Carl Sopchak]

On Monday, September 09, 2002 12:15 pm, Carl Sopchak wrote:
> > > Hummm...  I applied the patch, but I still don't have the main menu in
> > > QuickBooks....
> >
> > Hmm well, has that really been caused by my patch then ?
> > Did you remove my first patch to test it ?
>
> Oh, I just figured that the patch would take care of things like that. 
> I'll give it a try...
>
> Carl

AH HA!

It seems that the call to SetWindowLongA to unregister menu on owning window 
(per the comments; in the patch committed to CVS it's at "@@ -3842,6 +3844,9 
@@") is the culprit.  The committed patch had this call done unconditionally.  
The patch that you posted a few days ago made the call conditional on 
(lppop->hWnd).  When I commented this out (the condition and call), the menu 
"re-appears".  So, I guess for now, I'll leave the call out...

Andi, should I post a patch, or are you going to work on this some more?

Carl