Added wxEnableTopLevelWindows(bool enable) for use by wxSafeYield().

[wxWidgets.git] / src / common / event.cpp

diff --git a/src/common/event.cpp b/src/common/event.cpp
index 35f3a6593a84581d5779c3d60eb41989025b8caa..904178bc7b6445666a2d30d3c60c18c83d7e64ce 100644 (file)
--- a/src/common/event.cpp
+++ b/src/common/event.cpp
@@ -6,63 +6,82 @@
 // Created:     01/02/97
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart and Markus Holzem
 // Created:     01/02/97
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart and Markus Holzem

-// Licence:    wxWindows licence
+// Licence:       wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__

-#pragma implementation "event.h"
+    #pragma implementation "event.h"

 #endif
 
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
 #endif
 
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__

-#pragma hdrstop
+    #pragma hdrstop

 #endif
 
 #ifndef WX_PRECOMP
 #endif
 
 #ifndef WX_PRECOMP

-#include "wx/defs.h"
-#include "wx/control.h"
-#include "wx/utils.h"
-#include "wx/app.h"
-#include "wx/dc.h"
+    #include "wx/defs.h"
+    #include "wx/control.h"
+    #include "wx/utils.h"
+    #include "wx/app.h"
+    #include "wx/dc.h"

 #endif
 
 #include "wx/event.h"
 #include "wx/validate.h"
 
 #if !USE_SHARED_LIBRARY
 #endif
 
 #include "wx/event.h"
 #include "wx/validate.h"
 
 #if !USE_SHARED_LIBRARY

-IMPLEMENT_DYNAMIC_CLASS(wxEvtHandler, wxObject)
-IMPLEMENT_ABSTRACT_CLASS(wxEvent, wxObject)
-IMPLEMENT_DYNAMIC_CLASS(wxCommandEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxScrollEvent, wxCommandEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxMouseEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxKeyEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxSizeEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxPaintEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxEraseEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxMoveEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxFocusEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxCloseEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxShowEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxMaximizeEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxIconizeEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxMenuEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxJoystickEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxDropFilesEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxActivateEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxInitDialogEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxSysColourChangedEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxIdleEvent, wxEvent)
-IMPLEMENT_DYNAMIC_CLASS(wxUpdateUIEvent, wxEvent)
-
-const wxEventTable *wxEvtHandler::GetEventTable() const { return &wxEvtHandler::sm_eventTable; }
-
-const wxEventTable wxEvtHandler::sm_eventTable =
-       { NULL, &wxEvtHandler::sm_eventTableEntries[0] };
-
-const wxEventTableEntry wxEvtHandler::sm_eventTableEntries[] = { { 0, 0, 0, NULL } };
-
+    IMPLEMENT_DYNAMIC_CLASS(wxEvtHandler, wxObject)
+    IMPLEMENT_ABSTRACT_CLASS(wxEvent, wxObject)
+    IMPLEMENT_DYNAMIC_CLASS(wxCommandEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxNotifyEvent, wxCommandEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxScrollEvent, wxCommandEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxMouseEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxKeyEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxSizeEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxPaintEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxEraseEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxMoveEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxFocusEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxCloseEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxShowEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxMaximizeEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxIconizeEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxMenuEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxJoystickEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxDropFilesEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxActivateEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxInitDialogEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxSysColourChangedEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxIdleEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxUpdateUIEvent, wxCommandEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxNavigationKeyEvent, wxCommandEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxPaletteChangedEvent, wxEvent)
+    IMPLEMENT_DYNAMIC_CLASS(wxQueryNewPaletteEvent, wxEvent)
+
+    const wxEventTable *wxEvtHandler::GetEventTable() const
+        { return &wxEvtHandler::sm_eventTable; }
+
+    const wxEventTable wxEvtHandler::sm_eventTable =
+        { (const wxEventTable *)NULL, &wxEvtHandler::sm_eventTableEntries[0] };
+
+    const wxEventTableEntry wxEvtHandler::sm_eventTableEntries[] =
+        { { 0, 0, 0,
+    #ifdef __SGI_CC__
+        // stupid SGI compiler --- offer aug 98
+        0L }
+    #else // !SGI CC
+        NULL }
+    #endif // SGI/!SGI
+        };
+
+#endif // !USE_SHARED_LIBRARY
+
+#if wxUSE_THREADS
+/* To put pending event handlers */
+extern wxList *wxPendingEvents;
+extern wxCriticalSection *wxPendingEventsLocker;

 #endif
 
 /*
 #endif
 
 /*


@@ -76,13 +95,14 @@ const wxEventTableEntry wxEvtHandler::sm_eventTableEntries[] = { { 0, 0, 0, NULL
 
 wxEvent::wxEvent(int theId)
 {
 
 wxEvent::wxEvent(int theId)
 {

-  m_eventType = wxEVT_NULL;
-  m_eventObject = NULL;
-  m_eventHandle = NULL;
-  m_timeStamp = 0;
-  m_id = theId;
-  m_skipped = FALSE;
-  m_callbackUserData = NULL;
+    m_eventType = wxEVT_NULL;
+    m_eventObject = (wxObject *) NULL;
+    m_eventHandle = (char *) NULL;
+    m_timeStamp = 0;
+    m_id = theId;
+    m_skipped = FALSE;
+    m_callbackUserData = (wxObject *) NULL;
+    m_isCommandEvent = FALSE;

 }
 
 /*
 }
 
 /*


@@ -92,23 +112,28 @@ wxEvent::wxEvent(int theId)
 
 wxCommandEvent::wxCommandEvent(wxEventType commandType, int theId)
 {
 
 wxCommandEvent::wxCommandEvent(wxEventType commandType, int theId)
 {

-  m_eventType = commandType;
-  m_clientData = NULL;
-  m_extraLong = 0;
-  m_commandInt = 0;
-  m_id = theId;
-  m_commandString = NULL;
+    m_eventType = commandType;
+    m_clientData = (char *) NULL;
+    m_clientObject = (wxClientData *) NULL;
+    m_extraLong = 0;
+    m_commandInt = 0;
+    m_id = theId;
+    m_commandString = (char *) NULL;
+    m_isCommandEvent = TRUE;

 }
 
 /*
  * Scroll events
  */
 
 }
 
 /*
  * Scroll events
  */
 

-wxScrollEvent::wxScrollEvent(wxEventType commandType, int id, int pos, int orient):
-  wxCommandEvent(commandType, id)
+wxScrollEvent::wxScrollEvent(wxEventType commandType,
+                             int id,
+                             int pos,
+                             int orient)
+             : wxCommandEvent(commandType, id)

 {
 {

-  m_extraLong = orient;
-  m_commandInt = pos;
+    m_extraLong = orient;
+    m_commandInt = pos;

 }
 
 
 }
 
 


@@ -119,110 +144,122 @@ wxScrollEvent::wxScrollEvent(wxEventType commandType, int id, int pos, int orien
 
 wxMouseEvent::wxMouseEvent(wxEventType commandType)
 {
 
 wxMouseEvent::wxMouseEvent(wxEventType commandType)
 {

-  m_eventType = commandType;
-  m_metaDown = FALSE;
-  m_altDown = FALSE;
-  m_controlDown = FALSE;
-  m_shiftDown = FALSE;
+    m_eventType = commandType;
+    m_metaDown = FALSE;
+    m_altDown = FALSE;
+    m_controlDown = FALSE;
+    m_shiftDown = FALSE;
+    m_leftDown = FALSE;
+    m_rightDown = FALSE;
+    m_middleDown = FALSE;
+    m_x = 0;
+    m_y = 0;

 }
 
 // True if was a button dclick event (1 = left, 2 = middle, 3 = right)
 // or any button dclick event (but = -1)
 bool wxMouseEvent::ButtonDClick(int but) const
 {
 }
 
 // True if was a button dclick event (1 = left, 2 = middle, 3 = right)
 // or any button dclick event (but = -1)
 bool wxMouseEvent::ButtonDClick(int but) const
 {

-  switch (but) {
-    case -1:
-      return (LeftDClick() || MiddleDClick() || RightDClick());
-    case 1:
-      return LeftDClick();
-    case 2:
-      return MiddleDClick();
-    case 3:
-      return RightDClick();
-    default:
-      return FALSE;
-  }
-  // NOTREACHED
+    switch (but)
+    {
+        case -1:
+            return (LeftDClick() || MiddleDClick() || RightDClick());
+        case 1:
+            return LeftDClick();
+        case 2:
+            return MiddleDClick();
+        case 3:
+            return RightDClick();
+        default:
+            wxFAIL_MSG(_T("invalid parameter in wxMouseEvent::ButtonDClick"));
+    }
+
+    return FALSE;

 }
 
 // True if was a button down event (1 = left, 2 = middle, 3 = right)
 // or any button down event (but = -1)
 bool wxMouseEvent::ButtonDown(int but) const
 {
 }
 
 // True if was a button down event (1 = left, 2 = middle, 3 = right)
 // or any button down event (but = -1)
 bool wxMouseEvent::ButtonDown(int but) const
 {

-  switch (but) {
-    case -1:
-      return (LeftDown() || MiddleDown() || RightDown());
-    case 1:
-      return LeftDown();
-    case 2:
-      return MiddleDown();
-    case 3:
-      return RightDown();
-    default:
-      return FALSE;
-  }
-  // NOTREACHED
+    switch (but)
+    {
+        case -1:
+            return (LeftDown() || MiddleDown() || RightDown());
+        case 1:
+            return LeftDown();
+        case 2:
+            return MiddleDown();
+        case 3:
+            return RightDown();
+        default:
+            wxFAIL_MSG(_T("invalid parameter in wxMouseEvent::ButtonDown"));
+    }
+
+    return FALSE;

 }
 
 // True if was a button up event (1 = left, 2 = middle, 3 = right)
 // or any button up event (but = -1)
 bool wxMouseEvent::ButtonUp(int but) const
 {
 }
 
 // True if was a button up event (1 = left, 2 = middle, 3 = right)
 // or any button up event (but = -1)
 bool wxMouseEvent::ButtonUp(int but) const
 {

-  switch (but) {
-    case -1:
-      return (LeftUp() || MiddleUp() || RightUp());
-    case 1:
-      return LeftUp();
-    case 2:
-      return MiddleUp();
-    case 3:
-      return RightUp();
-    default:
-      return FALSE;
-  }
-  // NOTREACHED
+    switch (but) {
+        case -1:
+            return (LeftUp() || MiddleUp() || RightUp());
+        case 1:
+            return LeftUp();
+        case 2:
+            return MiddleUp();
+        case 3:
+            return RightUp();
+        default:
+            wxFAIL_MSG(_T("invalid parameter in wxMouseEvent::ButtonUp"));
+    }
+
+    return FALSE;

 }
 
 // True if the given button is currently changing state
 bool wxMouseEvent::Button(int but) const
 {
 }
 
 // True if the given button is currently changing state
 bool wxMouseEvent::Button(int but) const
 {

-  switch (but) {
-    case -1:
-      return (ButtonUp(-1) || ButtonDown(-1) || ButtonDClick(-1)) ;
-    case 1:
-      return (LeftDown() || LeftUp() || LeftDClick());
-    case 2:
-      return (MiddleDown() || MiddleUp() || MiddleDClick());
-    case 3:
-      return (RightDown() || RightUp() || RightDClick());
-    default:
-      return FALSE;
-  }
-  // NOTREACHED
+    switch (but) {
+        case -1:
+            return (ButtonUp(-1) || ButtonDown(-1) || ButtonDClick(-1)) ;
+        case 1:
+            return (LeftDown() || LeftUp() || LeftDClick());
+        case 2:
+            return (MiddleDown() || MiddleUp() || MiddleDClick());
+        case 3:
+            return (RightDown() || RightUp() || RightDClick());
+        default:
+            wxFAIL_MSG(_T("invalid parameter in wxMouseEvent::Button"));
+    }
+
+    return FALSE;

 }
 
 bool wxMouseEvent::ButtonIsDown(int but) const
 {
 }
 
 bool wxMouseEvent::ButtonIsDown(int but) const
 {

-  switch (but) {
-    case -1:
-      return (LeftIsDown() || MiddleIsDown() || RightIsDown());
-    case 1:
-      return LeftIsDown();
-    case 2:
-      return MiddleIsDown();
-    case 3:
-      return RightIsDown();
-    default:
-      return FALSE;
-  }
-  // NOTREACHED
+    switch (but) {
+        case -1:
+            return (LeftIsDown() || MiddleIsDown() || RightIsDown());
+        case 1:
+            return LeftIsDown();
+        case 2:
+            return MiddleIsDown();
+        case 3:
+            return RightIsDown();
+        default:
+            wxFAIL_MSG(_T("invalid parameter in wxMouseEvent::ButtonIsDown"));
+    }
+
+    return FALSE;

 }
 
 // Find the logical position of the event given the DC
 wxPoint wxMouseEvent::GetLogicalPosition(const wxDC& dc) const
 {
 }
 
 // Find the logical position of the event given the DC
 wxPoint wxMouseEvent::GetLogicalPosition(const wxDC& dc) const
 {

-       wxPoint pt(dc.DeviceToLogicalX(m_x), dc.DeviceToLogicalY(m_y));
-       return pt;
+    wxPoint pt(dc.DeviceToLogicalX(m_x), dc.DeviceToLogicalY(m_y));
+    return pt;

 }
 
 
 }
 
 


@@ -233,49 +270,99 @@ wxPoint wxMouseEvent::GetLogicalPosition(const wxDC& dc) const
 
 wxKeyEvent::wxKeyEvent(wxEventType type)
 {
 
 wxKeyEvent::wxKeyEvent(wxEventType type)
 {

-  m_eventType = type;
-  m_shiftDown = FALSE;
-  m_controlDown = FALSE;
-  m_metaDown = FALSE;
-  m_altDown = FALSE;
-  m_keyCode = 0;
+    m_eventType = type;
+    m_shiftDown = FALSE;
+    m_controlDown = FALSE;
+    m_metaDown = FALSE;
+    m_altDown = FALSE;
+    m_keyCode = 0;

 }
 
 /*
  * Event handler
  */
 
 }
 
 /*
  * Event handler
  */
 

-wxEvtHandler::wxEvtHandler(void)
+wxEvtHandler::wxEvtHandler()

 {
 {

-  m_clientData = NULL;
-  m_nextHandler = NULL;
-  m_previousHandler = NULL;
-  m_enabled = TRUE;
-  m_dynamicEvents = NULL;
+    m_nextHandler = (wxEvtHandler *) NULL;
+    m_previousHandler = (wxEvtHandler *) NULL;
+    m_enabled = TRUE;
+    m_dynamicEvents = (wxList *) NULL;
+    m_isWindow = FALSE;
+#if wxUSE_THREADS
+    m_eventsLocker = new wxCriticalSection();
+#endif
+    m_pendingEvents = (wxList *) NULL;

 }
 
 }
 

-wxEvtHandler::~wxEvtHandler(void)
+wxEvtHandler::~wxEvtHandler()

 {
 {

-  // Takes itself out of the list of handlers
-  if (m_previousHandler)
-    m_previousHandler->m_nextHandler = m_nextHandler;
-
-  if (m_nextHandler)
-    m_nextHandler->m_previousHandler = m_previousHandler;
-    
-  if (m_dynamicEvents)
-  {
-    wxNode *node = m_dynamicEvents->First();
-    while (node)
+    // Takes itself out of the list of handlers
+    if (m_previousHandler)
+        m_previousHandler->m_nextHandler = m_nextHandler;
+
+    if (m_nextHandler)
+        m_nextHandler->m_previousHandler = m_previousHandler;
+
+    if (m_dynamicEvents)

     {
     {

-      wxEventTableEntry *entry = (wxEventTableEntry*)node->Data();
-      if (entry->m_callbackUserData) delete entry->m_callbackUserData;
-      delete entry;
-      node = node->Next();
+        wxNode *node = m_dynamicEvents->First();
+        while (node)
+        {
+            wxEventTableEntry *entry = (wxEventTableEntry*)node->Data();
+            if (entry->m_callbackUserData) delete entry->m_callbackUserData;
+            delete entry;
+            node = node->Next();
+        }
+        delete m_dynamicEvents;
+    };
+
+    if (m_pendingEvents)
+      delete m_pendingEvents;
+
+#if wxUSE_THREADS
+    delete m_eventsLocker;
+#endif
+}
+
+#if wxUSE_THREADS
+bool wxEvtHandler::ProcessThreadEvent(wxEvent& event)
+{
+    wxEvent *event_main;
+    wxCriticalSectionLocker locker(*m_eventsLocker);
+
+    // check that we are really in a child thread
+    wxASSERT( !wxThread::IsMain() );
+
+    if (m_pendingEvents == NULL)
+      m_pendingEvents = new wxList();
+
+    event_main = (wxEvent *)event.GetClassInfo()->CreateObject();
+    *event_main = event;
+
+    m_pendingEvents->Append(event_main);
+
+    wxPendingEventsLocker->Enter();
+    wxPendingEvents->Append(this);
+    wxPendingEventsLocker->Leave();
+
+    return TRUE;
+}
+
+void wxEvtHandler::ProcessPendingEvents()
+{
+    wxCriticalSectionLocker locker(*m_eventsLocker);
+    wxNode *node = m_pendingEvents->First();
+    wxEvent *event;
+
+    while (node != NULL) {
+      event = (wxEvent *)node->Data();
+      ProcessEvent(*event);
+      delete node;
+      node = m_pendingEvents->First();

     }
     }

-    delete m_dynamicEvents;
-  };

 }
 }

+#endif

 
 /*
  * Event table stuff
 
 /*
  * Event table stuff


@@ -283,240 +370,211 @@ wxEvtHandler::~wxEvtHandler(void)
 
 bool wxEvtHandler::ProcessEvent(wxEvent& event)
 {
 
 bool wxEvtHandler::ProcessEvent(wxEvent& event)
 {

-  // An event handler can be enabled or disabled
-  if ( GetEvtHandlerEnabled() )
-  {
-    // Handle per-instance dynamic event tables first
-  
-    if (SearchDynamicEventTable( event )) return TRUE;
-
-    // Then static per-class event tables  
-  
-    const wxEventTable *table = GetEventTable();
-
-    // Try the associated validator first, if this is a window.
-    // Problem: if the event handler of the window has been replaced,
-    // this wxEvtHandler may no longer be a window.
-    // Therefore validators won't be processed if the handler
-    // has been replaced with SetEventHandler.
-    // THIS CAN BE CURED if PushEventHandler is used instead of
-    // SetEventHandler, and then processing will be passed down the
-    // chain of event handlers.
-    if (IsKindOf(CLASSINFO(wxWindow)))
-    {
-         wxWindow *win = (wxWindow *)this;
-
-         // Can only use the validator of the window which
-         // is receiving the event
-         if ( (win == event.GetEventObject()) &&
-             win->GetValidator() &&
-                 win->GetValidator()->ProcessEvent(event))
-               return TRUE;
-    }
+    // check that our flag corresponds to reality
+    wxASSERT( m_isWindow == IsKindOf(CLASSINFO(wxWindow)) );

 
 

-    // Search upwards through the inheritance hierarchy
-    while (table)
+    // An event handler can be enabled or disabled
+    if ( GetEvtHandlerEnabled() )

     {
     {

-      if (SearchEventTable((wxEventTable&)*table, event))
-          return TRUE;
-      table = table->baseTable;
+#if wxUSE_THREADS
+       // Check whether we are in a child thread.
+        if (!wxThread::IsMain())
+          return ProcessThreadEvent(event);
+#endif
+        // Handle per-instance dynamic event tables first
+
+        if ( m_dynamicEvents && SearchDynamicEventTable(event) )
+            return TRUE;
+
+        // Then static per-class event tables
+
+        const wxEventTable *table = GetEventTable();
+
+        // Try the associated validator first, if this is a window.
+        // Problem: if the event handler of the window has been replaced,
+        // this wxEvtHandler may no longer be a window.
+        // Therefore validators won't be processed if the handler
+        // has been replaced with SetEventHandler.
+        // THIS CAN BE CURED if PushEventHandler is used instead of
+        // SetEventHandler, and then processing will be passed down the
+        // chain of event handlers.
+        if ( m_isWindow )
+        {
+            wxWindow *win = (wxWindow *)this;
+
+            // Can only use the validator of the window which
+            // is receiving the event
+            if ( win == event.GetEventObject() )
+            {
+                wxValidator *validator = win->GetValidator();
+                if ( validator && validator->ProcessEvent(event) )
+                {
+                    return TRUE;
+                }
+            }
+        }
+
+        // Search upwards through the inheritance hierarchy
+        while ( table )
+        {
+            if ( SearchEventTable((wxEventTable&)*table, event) )
+                return TRUE;
+            table = table->baseTable;
+        }

     }
     }

-  }
-
-  // Try going down the event handler chain
-  if ( GetNextHandler() )
-  {
-           if ( GetNextHandler()->ProcessEvent(event) )
-                       return TRUE;
-  }
-
-  // Carry on up the parent-child hierarchy,
-  // but only if event is a command event: it wouldn't
-  // make sense for a parent to receive a child's size event, for example
-  if (IsKindOf(CLASSINFO(wxWindow)) && event.IsKindOf(CLASSINFO(wxCommandEvent)))
-  {
-    wxWindow *win = (wxWindow *)this;
-       wxWindow *parent = win->GetParent();
-    if (parent && !parent->IsBeingDeleted())
-      return win->GetParent()->GetEventHandler()->ProcessEvent(event);
-  }
-
-  // Last try - application object
-  if (wxTheApp && this != wxTheApp && wxTheApp->ProcessEvent(event))
-    return TRUE;
-  else
-    return FALSE;
-}

 
 

-bool wxEvtHandler::SearchEventTable(wxEventTable& table, wxEvent& event)
-{
-  int i = 0;
-  int commandId = event.GetId();
-  
-  while (table.entries[i].m_fn != NULL)
-  {
-    wxEventType eventType = (wxEventType) table.entries[i].m_eventType;
-
-    if ((event.GetEventType() == table.entries[i].m_eventType) &&
-        (table.entries[i].m_id == -1 || // Match, if event spec says any id will do (id == -1)
-          (table.entries[i].m_lastId == -1 && commandId == table.entries[i].m_id) ||
-          (table.entries[i].m_lastId != -1 &&
-            (commandId >= table.entries[i].m_id && commandId <= table.entries[i].m_lastId))))
+    // Try going down the event handler chain
+    if ( GetNextHandler() )

     {
     {

-               event.Skip(FALSE);
-                event.m_callbackUserData = table.entries[i].m_callbackUserData;
-       
-        (this->*((wxEventFunction) (table.entries[i].m_fn)))(event);
-
-               if ( event.GetSkipped() )
-                       return FALSE;
-                 else
+        if ( GetNextHandler()->ProcessEvent(event) )

             return TRUE;
     }
             return TRUE;
     }

-    i ++;
-  }
-  return FALSE;
-}

 
 

-void wxEvtHandler::Connect( int id, int lastId,
-                            int eventType,
-                           wxObjectEventFunction func,
-                           wxObject *userData )
-{
-  wxEventTableEntry *entry = new wxEventTableEntry;
-  entry->m_id = id;
-  entry->m_lastId = lastId;
-  entry->m_eventType = eventType;
-  entry->m_fn = func;
-  entry->m_callbackUserData = userData;
-  
-  if (!m_dynamicEvents) 
-    m_dynamicEvents = new wxList;
-    
-  m_dynamicEvents->Append( (wxObject*) entry );
-}
-
-bool wxEvtHandler::SearchDynamicEventTable( wxEvent& event )
-{
-  if (!m_dynamicEvents) return FALSE;
-  
-  int commandId = event.GetId();
-
-  wxNode *node = m_dynamicEvents->First();
-  while (node)
-  {
-    wxEventTableEntry *entry = (wxEventTableEntry*)node->Data();
-    wxEventType eventType = (wxEventType) entry->m_eventType;
-  
-    if (entry->m_fn)
+    // Carry on up the parent-child hierarchy,
+    // but only if event is a command event: it wouldn't
+    // make sense for a parent to receive a child's size event, for example
+    if ( m_isWindow && event.IsCommandEvent() )

     {
     {

-    if ((event.GetEventType() == entry->m_eventType) &&
-        (entry->m_id == -1 ||     // Match, if event spec says any id will do (id == -1)
-        (entry->m_lastId == -1 && commandId == entry->m_id) ||
-        (entry->m_lastId != -1 &&
-        (commandId >= entry->m_id && commandId <= entry->m_lastId))))
-       {
-         event.Skip(FALSE);
-          event.m_callbackUserData = entry->m_callbackUserData;
-       
-          (this->*((wxEventFunction) (entry->m_fn)))(event);
-
-         if (event.GetSkipped()) 
-           return FALSE;
-         else
-            return TRUE;
-       }
-    };
-    node = node->Next();
-  }
-  return FALSE;
-};
+        wxWindow *win = (wxWindow *)this;
+        wxWindow *parent = win->GetParent();
+        if (parent && !parent->IsBeingDeleted())
+            return parent->GetEventHandler()->ProcessEvent(event);
+    }

 
 

-/*
-#if WXWIN_COMPATIBILITY
-void wxEvtHandler::OldOnMenuCommand(int cmd)
-{
-    if (GetNextHandler()) GetNextHandler()->OldOnMenuCommand(cmd);
-}
+    // Last try - application object.
+    if ( wxTheApp && (this != wxTheApp) )
+    {
+        // Special case: don't pass wxEVT_IDLE to wxApp, since it'll always
+        // swallow it. wxEVT_IDLE is sent explicitly to wxApp so it will be
+        // processed appropriately via SearchEventTable.
+        if ( event.GetEventType() != wxEVT_IDLE )
+        {
+            if ( wxTheApp->ProcessEvent(event) )
+                return TRUE;
+        }
+    }

 
 

-void wxEvtHandler::OldOnMenuSelect(int cmd)
-{
-    if (GetNextHandler()) GetNextHandler()->OldOnMenuSelect(cmd);
+    return FALSE;

 }
 
 }
 

-void wxEvtHandler::OldOnInitMenuPopup(int pos)
+bool wxEvtHandler::SearchEventTable(wxEventTable& table, wxEvent& event)

 {
 {

-    if (GetNextHandler()) GetNextHandler()->OldOnInitMenuPopup(pos);
-}
+    int i = 0;
+    int commandId = event.GetId();

 
 

-void wxEvtHandler::OldOnScroll(wxCommandEvent& event)
-{
-    if (GetNextHandler()) GetNextHandler()->OldOnScroll(event);
-}
+    // BC++ doesn't like while (table.entries[i].m_fn)

 
 

-void wxEvtHandler::OldOnPaint(void)
-{
-    if (GetNextHandler()) GetNextHandler()->OldOnPaint();
-}
-void wxEvtHandler::OldOnSize(int width, int height)
-{
-    if (GetNextHandler()) GetNextHandler()->OldOnSize(width, height);
+#ifdef __SC__
+    while (table.entries[i].m_fn != 0)
+#else
+    while (table.entries[i].m_fn != 0L)
+#endif
+    {
+        if ((event.GetEventType() == table.entries[i].m_eventType) &&
+                (table.entries[i].m_id == -1 || // Match, if event spec says any id will do (id == -1)
+                 (table.entries[i].m_lastId == -1 && commandId == table.entries[i].m_id) ||
+                 (table.entries[i].m_lastId != -1 &&
+                  (commandId >= table.entries[i].m_id && commandId <= table.entries[i].m_lastId))))
+        {
+            event.Skip(FALSE);
+            event.m_callbackUserData = table.entries[i].m_callbackUserData;
+
+            (this->*((wxEventFunction) (table.entries[i].m_fn)))(event);
+
+            if ( event.GetSkipped() )
+                return FALSE;
+            else
+                return TRUE;
+        }
+        i++;
+    }
+    return FALSE;

 }
 }

-
-void wxEvtHandler::OldOnMove(int x, int y)
+void wxEvtHandler::Connect( int id, int lastId,
+                            wxEventType eventType,
+                            wxObjectEventFunction func,
+                            wxObject *userData )

 {
 {

-    if (GetNextHandler()) GetNextHandler()->OldOnMove(x, y);
-}
+    wxEventTableEntry *entry = new wxEventTableEntry;
+    entry->m_id = id;
+    entry->m_lastId = lastId;
+    entry->m_eventType = eventType;
+    entry->m_fn = func;
+    entry->m_callbackUserData = userData;

 
 

-void wxEvtHandler::OldOnMouseEvent(wxMouseEvent& event)
-{
-    if (GetNextHandler()) GetNextHandler()->OldOnMouseEvent(event);
-}
+    if (!m_dynamicEvents)
+        m_dynamicEvents = new wxList;

 
 

-void wxEvtHandler::OldOnChar(wxKeyEvent& event)
-{
-    if (GetNextHandler()) GetNextHandler()->OldOnChar(event);
+    m_dynamicEvents->Append( (wxObject*) entry );

 }
 
 }
 

-// Under Windows, we can intercept character input per dialog or frame
-bool wxEvtHandler::OldOnCharHook(wxKeyEvent& event)
+bool wxEvtHandler::SearchDynamicEventTable( wxEvent& event )

 {
 {

-    if (GetNextHandler()) return GetNextHandler()->OldOnCharHook(event);
-       else return FALSE;
-}
+    wxCHECK_MSG( m_dynamicEvents, FALSE,
+                 _T("caller should check that we have dynamic events") );

 
 

-void wxEvtHandler::OldOnActivate(bool active)
-{
-    if (GetNextHandler()) GetNextHandler()->OldOnActivate(active);
-}
+    int commandId = event.GetId();

 
 

-void wxEvtHandler::OldOnSetFocus(void)
-{
-    if (GetNextHandler()) GetNextHandler()->OldOnSetFocus();
-}
+    wxNode *node = m_dynamicEvents->First();
+    while (node)
+    {
+        wxEventTableEntry *entry = (wxEventTableEntry*)node->Data();
+
+        if (entry->m_fn)
+        {
+            // Match, if event spec says any id will do (id == -1)
+            if ( (event.GetEventType() == entry->m_eventType) &&
+                 (entry->m_id == -1 ||
+                  (entry->m_lastId == -1 && commandId == entry->m_id) ||
+                  (entry->m_lastId != -1 &&
+                  (commandId >= entry->m_id && commandId <= entry->m_lastId))) )
+            {
+                event.Skip(FALSE);
+                event.m_callbackUserData = entry->m_callbackUserData;
+
+                (this->*((wxEventFunction) (entry->m_fn)))(event);
+
+                if (event.GetSkipped())
+                    return FALSE;
+                else
+                    return TRUE;
+            }
+        }
+        node = node->Next();
+    }
+    return FALSE;
+};

 
 

-void wxEvtHandler::OldOnKillFocus(void)
+#if WXWIN_COMPATIBILITY
+bool wxEvtHandler::OnClose()

 {
 {

-    if (GetNextHandler()) GetNextHandler()->OldOnKillFocus();
+    if (GetNextHandler())
+        return GetNextHandler()->OnClose();
+    else
+        return FALSE;

 }
 }

+#endif // WXWIN_COMPATIBILITY

 
 

-bool wxEvtHandler::OldOnSysColourChange(void)
+// Find a window with the focus, that is also a descendant of the given window.
+// This is used to determine the window to initially send commands to.
+wxWindow* wxFindFocusDescendant(wxWindow* ancestor)

 {
 {

-    if (GetNextHandler()) return GetNextHandler()->OldOnSysColourChange();
-    return FALSE;
-}
+    // Process events starting with the window with the focus, if any.
+    wxWindow* focusWin = wxWindow::FindFocus();
+    wxWindow* win = focusWin;

 
 

-void wxEvtHandler::OldOnDropFiles(int n, char *files[], int x, int y)
-{
-    if (GetNextHandler()) GetNextHandler()->OldOnDropFiles(n, files, x, y);
-}
-#endif
-*/
+    // Check if this is a descendant of this frame.
+    // If not, win will be set to NULL.
+    while (win)
+    {
+        if (win == ancestor)
+            break;
+        else
+            win = win->GetParent();
+    }
+    if (win == (wxWindow*) NULL)
+        focusWin = (wxWindow*) NULL;

 
 

-bool wxEvtHandler::OnClose(void)
-{
-    if (GetNextHandler()) return GetNextHandler()->OnClose();
-    else return FALSE;
+    return focusWin;

 }
 
 }
 

-