Should return wxImage not bool

[wxWidgets.git] / src / cocoa / app.mm

diff --git a/src/cocoa/app.mm b/src/cocoa/app.mm
index e2336ca1b18ca3cfe568ff67499d0d743c1c31c5..59fcbc9f466a373d35211d4a9092b670e7991b20 100644 (file)
--- a/src/cocoa/app.mm
+++ b/src/cocoa/app.mm
@@ -71,13 +71,16 @@ WX_IMPLEMENT_POSER(wxPoserNSApplication);
     the queue to ensure this method will return.
     See wxEventLoop::Exit() for more information.
 
     the queue to ensure this method will return.
     See wxEventLoop::Exit() for more information.
 

- Ê ÊRN: We used to use nil as the untilDate in previous versions since nil
-Ê  Êis a shorter and more concise way of specifying an infinite amount of
-ÊÊ Êtime than [NSDate distantPast]. ÊHowever, Apple neglects to mention in
-Ê  Êtheir documentation that nil is not handled correctly in OSX 10.2
-Ê  Ê(and possibly lower) and when the call is reached the system comes to
-Ê  Êa screeching halt, therefore we need to specify [NSDate distantPast]
-Ê  Êexplicitly so that wxCocoa will work correctly in OSX 10.2.
+    This overridden method calls the superclass method with an untilDate
+    parameter that indicates nil should be returned if there are no pending
+    events.  That is, nextEventMatchingMask: should not wait for an event.
+    If nil is returned then idle event processing occurs until the user
+    does not request anymore idle events or until a real event comes through.
+
+    RN: Even though Apple documentation states that nil can be passed in place 
+    of [NSDate distantPast] in the untilDate parameter, this causes Jaguar (10.2)
+    to get stuck in some kind of loop deep within nextEventMatchingMask:, thus we 
+    need to explicitly pass [NSDate distantPast] instead.

 */
    
 - (NSEvent *)nextEventMatchingMask:(unsigned int)mask untilDate:(NSDate *)expiration inMode:(NSString *)mode dequeue:(BOOL)flag
 */
    
 - (NSEvent *)nextEventMatchingMask:(unsigned int)mask untilDate:(NSDate *)expiration inMode:(NSString *)mode dequeue:(BOOL)flag