Hi Georg,
thanks for reporting problems with the GModel code - I looked
at the recent changes to the NSMenuItem and I am still puzzled
by the fact that (AFAIK) in OPENSTEP the target of the submenuAction:
is the submenu to be detached, while in GNUstep it seems we are
using the menu itself (=the menu from which you detach the submenu).
I thought we were using the submenu in GNUstep too, but I seem
to be wrong ? We should switch to the submenu I think. We can
even spare an instance variable, since we can store the submenu in
the target ivar (according to an old comment I added to the gmodel code,
someone reported to me that this is how what happens on OPENSTEP).
There was a recent bug report on gnats too, in which the person reporting
the bug marked (as part of the solution) that the target should be the
submenu (and not the menu). I experimented with replacing the menu with the
submenu in my GNUstep and nothing happens - all works fine - it seems
submenuAction: is in reality never used in GNUstep ? Anyway I didn't
dare to make the switch without asking other developers if they agree;
if they do, we can start switching - I guess that could help fixing
the problem with gmodels - and probably also simplifying the gmodel code.
Btw, even ignoring what happens on OPENSTEP, it seems to me better
if the target of the submenuAction: is the menu to be detached.
Please Cc: any follow-up or reply to [EMAIL PROTECTED]
