bugfix for nil argumenmt to makefirstResponder: as reported by andreas hoschler

git-svn-id: svn+ssh://svn.gna.org/svn/gnustep/libs/gui/trunk@22860 72102866-910b-0410-8b05-ffd578937521

ChangeLog

@@ -1,3 +1,8 @@
+2006-05-05  Richard Frith-Macdonald <rfm@gnu.org>
+
+	* Source/NSWindow.m: ([-makeFirstResponder:]) fixed for nil argument.
+	* Headers/AppKit/NSWindow.h: documented method.
+
 2006-05-03  Fred Kiefer <FredKiefer@gmx.de>
 
 	* Headers/Additions/GNUstepGUI/GSDisplayServer.h (-nativeWindow:::::):

Headers/AppKit/NSWindow.h

@@ -393,6 +393,23 @@ style. */
 - (NSScreen*) screen;
 
 - (NSResponder*) firstResponder;
+
+/**
+ * This method attempts to make aResponder the first responder.<br />
+ * If aResponder is already the first responder, this method has no
+ * effect and simply returns YES.
+ * Otherwise, the method sends a -resignFirstResponder message to the
+ * current first responder (if there is one) and immediately returns NO if
+ * the current first responder refuses to resign.<br />
+ * Then the method asks aResponder to become first responder by sending
+ * it a -becomeFirstResponder message, and if that returns YES then this
+ * method immediately returns YES.<br />
+ * However, if that returns NO, the receiver is made the first responder by
+ * sending it a -becomeFirstResponder message, and this method returns NO.<br />
+ * If aResponder is neither nil nor an instance of NSResponder (or of a
+ * subclass of NSResponder) then behavior is undefined (though the current
+ * GNUstep implementation just returns NO).
+ */
 - (BOOL) makeFirstResponder: (NSResponder*)aResponder;
 
 /*

Source/NSWindow.m

@@ -2667,11 +2667,14 @@ resetCursorRectsForView(NSView *theView)
   if (_firstResponder == aResponder)
     return YES;
 
-  if (![aResponder isKindOfClass: responderClass])
+  if (aResponder != nil)
-    return NO;
+    {
+      if (![aResponder isKindOfClass: responderClass])
+	return NO;
 
-  if (![aResponder acceptsFirstResponder])
+      if (![aResponder acceptsFirstResponder])
-    return NO;
+	return NO;
+    }
 
   /* So that the implementation of -resignFirstResponder in
      _firstResponder might ask for what will be the new first