Re: [cp-patches] Patch: BasicMenuItemUI fix

[Roman Kennke]

Hi Lillian,

It looks a little strange to handle CheckBoxMenuItem specific cases in
the superclass. However, I understand that the API of
BasicCheckBoxMenuItem does not leave much room for specific handling of
this.

What looks a bit strange to me is that the specs define a method
processMouseEvent in BasicCheckBoxMenuItemUI and
BasicRadioButtonMenuItemUI but _not_ in one of the superclasses. Could
these method possibly have something to do with your problem? Can you
make any sense of these methods? Who is supposed to call these? The
MouseListener of the superclass? A private inner mouse listener? If it
was a private mouse listener, where should it be set up? These classes
don't override installListeners or something like that..

Very strange indeed.

/Roman

signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil