Hi all,

I believe I have posted this already a few weeks ago, but I could also be wrong. I just fall again over a little issue in NSWindow makeFirstResponder.

- (BOOL)makeFirstResponder:(NSResponder *)aResponder
{
  ...
  _firstResponder = aResponder;
  if (_firstResponder)  // <------------------- add this check
    {
     if (![_firstResponder becomeFirstResponder])
       {
        _firstResponder = self;
        [_firstResponder becomeFirstResponder];
        return NO;
       }
    }

  return YES;
}

Since I didn't remember what exactly the "if (_firstResponder)" check was good for I tried to get lucky without it after downlöoading latest trunk. But this one is really important as I have learned again just now. Otherwise makeFirstResponder returns a wrong value if

        [window makeFirstResponder:nil];

is called. If nothing speaks against this modification please apply it to trunk.

Thanks,

  Andreas



_______________________________________________
Discuss-gnustep mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/discuss-gnustep

Reply via email to