Updates for debian build scripts to suit new library and tool status.

[wxWidgets.git] / src / msw / window.cpp

diff --git a/src/msw/window.cpp b/src/msw/window.cpp
index 91116b67ac5a916a4c33c6aa49ff45f1125b9252..d3eab805ef5e40a838efd8b01f3277cd9731eb18 100644 (file)
--- a/src/msw/window.cpp
+++ b/src/msw/window.cpp
@@ -97,7 +97,7 @@
 
 #include <string.h>
 
 
 #include <string.h>
 

-#if (!defined(__GNUWIN32_OLD__) && !defined(__WXMICROWIN__) && !defined(__WXWINCE__)) || defined(__CYGWIN10__)
+#if (!defined(__GNUWIN32_OLD__) && !defined(__WXMICROWIN__) /* && !defined(__WXWINCE__) */ ) || defined(__CYGWIN10__)

     #include <shellapi.h>
     #include <mmsystem.h>
 #endif
     #include <shellapi.h>
     #include <mmsystem.h>
 #endif


@@ -106,7 +106,7 @@
     #include <windowsx.h>
 #endif
 
     #include <windowsx.h>
 #endif
 

-#if (!defined(__GNUWIN32_OLD__) && !defined(__WXMICROWIN__) && !defined(__WXWINCE__)) || defined(__CYGWIN10__)
+#if (!defined(__GNUWIN32_OLD__) && !defined(__WXMICROWIN__) /* && !defined(__WXWINCE__) */ ) || defined(__CYGWIN10__)

     #ifdef __WIN95__
         #include <commctrl.h>
     #endif
     #ifdef __WIN95__
         #include <commctrl.h>
     #endif


@@ -132,7 +132,7 @@ extern const wxChar *wxCanvasClassName;
 
 // true if we had already created the std colour map, used by
 // wxGetStdColourMap() and wxWindow::OnSysColourChanged()           (FIXME-MT)
 
 // true if we had already created the std colour map, used by
 // wxGetStdColourMap() and wxWindow::OnSysColourChanged()           (FIXME-MT)

-static bool gs_hasStdCmap = FALSE;
+static bool gs_hasStdCmap = false;

 
 // ---------------------------------------------------------------------------
 // private functions
 
 // ---------------------------------------------------------------------------
 // private functions


@@ -415,7 +415,7 @@ wxWindow *wxWindowMSW::FindItemByHWND(WXHWND hWnd, bool controlOnly) const
 // Default command handler
 bool wxWindowMSW::MSWCommand(WXUINT WXUNUSED(param), WXWORD WXUNUSED(id))
 {
 // Default command handler
 bool wxWindowMSW::MSWCommand(WXUINT WXUNUSED(param), WXWORD WXUNUSED(id))
 {

-    return FALSE;
+    return false;

 }
 
 // ----------------------------------------------------------------------------
 }
 
 // ----------------------------------------------------------------------------


@@ -425,12 +425,13 @@ bool wxWindowMSW::MSWCommand(WXUINT WXUNUSED(param), WXWORD WXUNUSED(id))
 void wxWindowMSW::Init()
 {
     // MSW specific
 void wxWindowMSW::Init()
 {
     // MSW specific

-    m_isBeingDeleted = FALSE;
+    m_isBeingDeleted = false;

     m_oldWndProc = NULL;
     m_oldWndProc = NULL;

-    m_mouseInWindow = FALSE;
-    m_lastKeydownProcessed = FALSE;
+    m_mouseInWindow = false;
+    m_lastKeydownProcessed = false;

 
     m_childrenDisabled = NULL;
 
     m_childrenDisabled = NULL;

+    m_frozenness = 0;

 
     // wxWnd
     m_hMenu = 0;
 
     // wxWnd
     m_hMenu = 0;


@@ -441,7 +442,7 @@ void wxWindowMSW::Init()
     m_yThumbSize = 0;
 
     // as all windows are created with WS_VISIBLE style...
     m_yThumbSize = 0;
 
     // as all windows are created with WS_VISIBLE style...

-    m_isShown = TRUE;
+    m_isShown = true;

 
 #if wxUSE_MOUSEEVENT_HACK
     m_lastMouseX =
 
 #if wxUSE_MOUSEEVENT_HACK
     m_lastMouseX =


@@ -453,7 +454,7 @@ void wxWindowMSW::Init()
 // Destructor
 wxWindowMSW::~wxWindowMSW()
 {
 // Destructor
 wxWindowMSW::~wxWindowMSW()
 {

-    m_isBeingDeleted = TRUE;
+    m_isBeingDeleted = true;

 
 #ifndef __WXUNIVERSAL__
     // VS: make sure there's no wxFrame with last focus set to us:
 
 #ifndef __WXUNIVERSAL__
     // VS: make sure there's no wxFrame with last focus set to us:


@@ -500,10 +501,10 @@ bool wxWindowMSW::Create(wxWindow *parent,
                          long style,
                          const wxString& name)
 {
                          long style,
                          const wxString& name)
 {

-    wxCHECK_MSG( parent, FALSE, wxT("can't create wxWindow without parent") );
+    wxCHECK_MSG( parent, false, wxT("can't create wxWindow without parent") );

 
     if ( !CreateBase(parent, id, pos, size, style, wxDefaultValidator, name) )
 
     if ( !CreateBase(parent, id, pos, size, style, wxDefaultValidator, name) )

-        return FALSE;
+        return false;

 
     parent->AddChild(this);
 
 
     parent->AddChild(this);
 


@@ -524,7 +525,7 @@ bool wxWindowMSW::Create(wxWindow *parent,
     if ( style & wxPOPUP_WINDOW )
     {
         msflags &= ~WS_VISIBLE;
     if ( style & wxPOPUP_WINDOW )
     {
         msflags &= ~WS_VISIBLE;

-        m_isShown = FALSE;
+        m_isShown = false;

     }
     else
     {
     }
     else
     {


@@ -596,7 +597,7 @@ wxWindow *wxWindowBase::FindFocus()
 bool wxWindowMSW::Enable(bool enable)
 {
     if ( !wxWindowBase::Enable(enable) )
 bool wxWindowMSW::Enable(bool enable)
 {
     if ( !wxWindowBase::Enable(enable) )

-        return FALSE;
+        return false;

 
     HWND hWnd = GetHwnd();
     if ( hWnd )
 
     HWND hWnd = GetHwnd();
     if ( hWnd )


@@ -606,7 +607,7 @@ bool wxWindowMSW::Enable(bool enable)
     // showing a modal dialog would result in total greying out (and ungreying
     // out later) of everything which would be really ugly
     if ( IsTopLevel() )
     // showing a modal dialog would result in total greying out (and ungreying
     // out later) of everything which would be really ugly
     if ( IsTopLevel() )

-        return TRUE;
+        return true;

 
     // when the parent is disabled, all of its children should be disabled as
     // well but when it is enabled back, only those of the children which
 
     // when the parent is disabled, all of its children should be disabled as
     // well but when it is enabled back, only those of the children which


@@ -655,13 +656,13 @@ bool wxWindowMSW::Enable(bool enable)
         m_childrenDisabled = NULL;
     }
 
         m_childrenDisabled = NULL;
     }
 

-    return TRUE;
+    return true;

 }
 
 bool wxWindowMSW::Show(bool show)
 {
     if ( !wxWindowBase::Show(show) )
 }
 
 bool wxWindowMSW::Show(bool show)
 {
     if ( !wxWindowBase::Show(show) )

-        return FALSE;
+        return false;

 
     HWND hWnd = GetHwnd();
     int cshow = show ? SW_SHOW : SW_HIDE;
 
     HWND hWnd = GetHwnd();
     int cshow = show ? SW_SHOW : SW_HIDE;


@@ -672,7 +673,7 @@ bool wxWindowMSW::Show(bool show)
         wxBringWindowToTop(hWnd);
     }
 
         wxBringWindowToTop(hWnd);
     }
 

-    return TRUE;
+    return true;

 }
 
 // Raise the window to the top of the Z order
 }
 
 // Raise the window to the top of the Z order


@@ -726,7 +727,7 @@ bool wxWindowMSW::SetFont(const wxFont& font)
     if ( !wxWindowBase::SetFont(font) )
     {
         // nothing to do
     if ( !wxWindowBase::SetFont(font) )
     {
         // nothing to do

-        return FALSE;
+        return false;

     }
 
     HWND hWnd = GetHwnd();
     }
 
     HWND hWnd = GetHwnd();


@@ -739,14 +740,14 @@ bool wxWindowMSW::SetFont(const wxFont& font)
         ::SendMessage(hWnd, WM_SETFONT, (WPARAM)hFont, MAKELPARAM(TRUE, 0));
     }
 
         ::SendMessage(hWnd, WM_SETFONT, (WPARAM)hFont, MAKELPARAM(TRUE, 0));
     }
 

-    return TRUE;
+    return true;

 }
 bool wxWindowMSW::SetCursor(const wxCursor& cursor)
 {
     if ( !wxWindowBase::SetCursor(cursor) )
     {
         // no change
 }
 bool wxWindowMSW::SetCursor(const wxCursor& cursor)
 {
     if ( !wxWindowBase::SetCursor(cursor) )
     {
         // no change

-        return FALSE;
+        return false;

     }
 
     if ( m_cursor.Ok() )
     }
 
     if ( m_cursor.Ok() )


@@ -763,7 +764,7 @@ bool wxWindowMSW::SetCursor(const wxCursor& cursor)
             ::SetCursor(GetHcursorOf(m_cursor));
     }
 
             ::SetCursor(GetHcursorOf(m_cursor));
     }
 

-    return TRUE;
+    return true;

 }
 
 void wxWindowMSW::WarpPointer (int x, int y)
 }
 
 void wxWindowMSW::WarpPointer (int x, int y)


@@ -974,19 +975,19 @@ void wxWindowMSW::SubclassWin(WXHWND hWnd)
 
     wxAssociateWinWithHandle(hwnd, this);
 
 
     wxAssociateWinWithHandle(hwnd, this);
 

-    m_oldWndProc = (WXFARPROC)::GetWindowLong((HWND)hWnd, GWL_WNDPROC);
+    m_oldWndProc = (WXFARPROC)wxGetWindowProc((HWND)hWnd);

 
     // we don't need to subclass the window of our own class (in the Windows
     // sense of the word)
        if ( !wxCheckWindowWndProc(hWnd, (WXFARPROC)wxWndProc) )
     {
 
     // we don't need to subclass the window of our own class (in the Windows
     // sense of the word)
        if ( !wxCheckWindowWndProc(hWnd, (WXFARPROC)wxWndProc) )
     {

-        ::SetWindowLong(hwnd, GWL_WNDPROC, (LONG) wxWndProc);
+        wxSetWindowProc(hwnd, wxWndProc);

     }
     else
     {
         // don't bother restoring it neither: this also makes it easy to
     }
     else
     {
         // don't bother restoring it neither: this also makes it easy to

-        // implement IsOfStandardClass() method which returns TRUE for the
-        // standard controls and FALSE for the wxWindows own windows as it can
+        // implement IsOfStandardClass() method which returns true for the
+        // standard controls and false for the wxWindows own windows as it can

         // simply check m_oldWndProc
         m_oldWndProc = NULL;
     }
         // simply check m_oldWndProc
         m_oldWndProc = NULL;
     }


@@ -1008,7 +1009,7 @@ void wxWindowMSW::UnsubclassWin()
         {
             if ( !wxCheckWindowWndProc((WXHWND)hwnd, m_oldWndProc) )
             {
         {
             if ( !wxCheckWindowWndProc((WXHWND)hwnd, m_oldWndProc) )
             {

-                ::SetWindowLong(hwnd, GWL_WNDPROC, (LONG) m_oldWndProc);
+                wxSetWindowProc(hwnd, (WNDPROC)m_oldWndProc);

             }
 
             m_oldWndProc = NULL;
             }
 
             m_oldWndProc = NULL;


@@ -1042,16 +1043,16 @@ bool wxCheckWindowWndProc(WXHWND hWnd, WXFARPROC wndProc)
                str == wxMDIChildFrameClassName ||
                str == wxMDIChildFrameClassNameNoRedraw ||
                str == _T("wxTLWHiddenParent"))
                str == wxMDIChildFrameClassName ||
                str == wxMDIChildFrameClassNameNoRedraw ||
                str == _T("wxTLWHiddenParent"))

-               return TRUE; // Effectively means don't subclass
+               return true; // Effectively means don't subclass

        else
        else

-               return FALSE;
+               return false;

 #else
     WNDCLASS cls;
     if ( !::GetClassInfo(wxGetInstance(), wxGetWindowClass(hWnd), &cls) )
     {
         wxLogLastError(_T("GetClassInfo"));
 
 #else
     WNDCLASS cls;
     if ( !::GetClassInfo(wxGetInstance(), wxGetWindowClass(hWnd), &cls) )
     {
         wxLogLastError(_T("GetClassInfo"));
 

-        return FALSE;
+        return false;

     }
 
     return wndProc == (WXFARPROC)cls.lpfnWndProc;
     }
 
     return wndProc == (WXFARPROC)cls.lpfnWndProc;


@@ -1224,7 +1225,7 @@ void wxWindowMSW::OnInternalIdle()
         if ( !IsMouseInWindow() )
         {
             // Generate a LEAVE event
         if ( !IsMouseInWindow() )
         {
             // Generate a LEAVE event

-            m_mouseInWindow = FALSE;
+            m_mouseInWindow = false;

 
             // Unfortunately the mouse button and keyboard state may have
             // changed by the time the OnInternalIdle function is called, so 'state'
 
             // Unfortunately the mouse button and keyboard state may have
             // changed by the time the OnInternalIdle function is called, so 'state'


@@ -1268,7 +1269,7 @@ void wxWindowMSW::OnInternalIdle()
 bool wxWindowMSW::Reparent(wxWindowBase *parent)
 {
     if ( !wxWindowBase::Reparent(parent) )
 bool wxWindowMSW::Reparent(wxWindowBase *parent)
 {
     if ( !wxWindowBase::Reparent(parent) )

-        return FALSE;
+        return false;

 
     HWND hWndChild = GetHwnd();
     HWND hWndParent = GetParent() ? GetWinHwnd(GetParent()) : (HWND)0;
 
     HWND hWndChild = GetHwnd();
     HWND hWndParent = GetParent() ? GetWinHwnd(GetParent()) : (HWND)0;


@@ -1282,7 +1283,7 @@ bool wxWindowMSW::Reparent(wxWindowBase *parent)
     }
 #endif // !__WXWINCE__
 
     }
 #endif // !__WXWINCE__
 

-    return TRUE;
+    return true;

 }
 
 static inline void SendSetRedraw(HWND hwnd, bool on)
 }
 
 static inline void SendSetRedraw(HWND hwnd, bool on)


@@ -1294,16 +1295,24 @@ static inline void SendSetRedraw(HWND hwnd, bool on)
 
 void wxWindowMSW::Freeze()
 {
 
 void wxWindowMSW::Freeze()
 {

-    SendSetRedraw(GetHwnd(), FALSE);
+    if ( !m_frozenness++ )
+    {
+        SendSetRedraw(GetHwnd(), false);
+    }

 }
 
 void wxWindowMSW::Thaw()
 {
 }
 
 void wxWindowMSW::Thaw()
 {

-    SendSetRedraw(GetHwnd(), TRUE);
+    wxASSERT_MSG( m_frozenness > 0, _T("Thaw() without matching Freeze()") );
+
+    if ( !--m_frozenness )
+    {
+        SendSetRedraw(GetHwnd(), true);

 
 

-    // we need to refresh everything or otherwise he invalidated area is not
-    // repainted
-    Refresh();
+        // we need to refresh everything or otherwise he invalidated area is not
+        // repainted
+        Refresh();
+    }

 }
 
 void wxWindowMSW::Refresh(bool eraseBack, const wxRect *rect)
 }
 
 void wxWindowMSW::Refresh(bool eraseBack, const wxRect *rect)


@@ -1754,7 +1763,7 @@ bool wxWindowMSW::DoPopupMenu(wxMenu *menu, int x, int y)
 
     menu->SetInvokingWindow(NULL);
 
 
     menu->SetInvokingWindow(NULL);
 

-    return TRUE;
+    return true;

 }
 
 #endif // wxUSE_MENUS_NATIVE
 }
 
 #endif // wxUSE_MENUS_NATIVE


@@ -1763,7 +1772,7 @@ bool wxWindowMSW::DoPopupMenu(wxMenu *menu, int x, int y)
 // pre/post message processing
 // ===========================================================================
 
 // pre/post message processing
 // ===========================================================================
 

-long wxWindowMSW::MSWDefWindowProc(WXUINT nMsg, WXWPARAM wParam, WXLPARAM lParam)
+WXLRESULT wxWindowMSW::MSWDefWindowProc(WXUINT nMsg, WXWPARAM wParam, WXLPARAM lParam)

 {
     if ( m_oldWndProc )
         return ::CallWindowProc(CASTWNDPROC m_oldWndProc, GetHwnd(), (UINT) nMsg, (WPARAM) wParam, (LPARAM) lParam);
 {
     if ( m_oldWndProc )
         return ::CallWindowProc(CASTWNDPROC m_oldWndProc, GetHwnd(), (UINT) nMsg, (WPARAM) wParam, (LPARAM) lParam);


@@ -1805,16 +1814,16 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
                 }
             }
 
                 }
             }
 

-            bool bForward = TRUE,
-                 bWindowChange = FALSE;
+            bool bForward = true,
+                 bWindowChange = false;

 
             // should we process this message specially?
 
             // should we process this message specially?

-            bool bProcess = TRUE;
+            bool bProcess = true;

             switch ( msg->wParam )
             {
                 case VK_TAB:
                     if ( lDlgCode & DLGC_WANTTAB ) {
             switch ( msg->wParam )
             {
                 case VK_TAB:
                     if ( lDlgCode & DLGC_WANTTAB ) {

-                        bProcess = FALSE;
+                        bProcess = false;

                     }
                     else {
                         // Ctrl-Tab cycles thru notebook pages
                     }
                     else {
                         // Ctrl-Tab cycles thru notebook pages


@@ -1826,15 +1835,34 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
                 case VK_UP:
                 case VK_LEFT:
                     if ( (lDlgCode & DLGC_WANTARROWS) || bCtrlDown )
                 case VK_UP:
                 case VK_LEFT:
                     if ( (lDlgCode & DLGC_WANTARROWS) || bCtrlDown )

-                        bProcess = FALSE;
+                        bProcess = false;

                     else
                     else

-                        bForward = FALSE;
+                        bForward = false;

                     break;
 
                 case VK_DOWN:
                 case VK_RIGHT:
                     if ( (lDlgCode & DLGC_WANTARROWS) || bCtrlDown )
                     break;
 
                 case VK_DOWN:
                 case VK_RIGHT:
                     if ( (lDlgCode & DLGC_WANTARROWS) || bCtrlDown )

-                        bProcess = FALSE;
+                        bProcess = false;
+                    break;
+
+                case VK_ESCAPE:
+                    {
+#if wxUSE_BUTTON
+                        wxButton *btn = wxDynamicCast(FindWindow(wxID_CANCEL),
+                                                      wxButton);
+                        if ( btn && btn->IsEnabled() )
+                        {
+                            // if we do have a cancel button, do press it
+                            btn->MSWCommand(BN_CLICKED, 0 /* unused */);
+
+                            // we consumed the message
+                            return true;
+                        }
+#endif // wxUSE_BUTTON
+
+                        bProcess = false;
+                    }

                     break;
 
                 case VK_RETURN:
                     break;
 
                 case VK_RETURN:


@@ -1844,7 +1872,7 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
                             // control wants to process Enter itself, don't
                             // call IsDialogMessage() which would interpret
                             // it
                             // control wants to process Enter itself, don't
                             // call IsDialogMessage() which would interpret
                             // it

-                            return FALSE;
+                            return false;

                         }
                         else if ( lDlgCode & DLGC_BUTTON )
                         {
                         }
                         else if ( lDlgCode & DLGC_BUTTON )
                         {


@@ -1860,7 +1888,7 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
                                     btn->MSWCommand(BN_CLICKED, 0 /* unused */);
                             }
 
                                     btn->MSWCommand(BN_CLICKED, 0 /* unused */);
                             }
 

-                            bProcess = FALSE;
+                            bProcess = false;

                         }
                         // FIXME: this should be handled by
                         //        wxNavigationKeyEvent handler and not here!!
                         }
                         // FIXME: this should be handled by
                         //        wxNavigationKeyEvent handler and not here!!


@@ -1874,7 +1902,7 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
                                 // if we do have a default button, do press it
                                 btn->MSWCommand(BN_CLICKED, 0 /* unused */);
 
                                 // if we do have a default button, do press it
                                 btn->MSWCommand(BN_CLICKED, 0 /* unused */);
 

-                                return TRUE;
+                                return true;

                             }
                             else // no default button
                             {
                             }
                             else // no default button
                             {


@@ -1887,13 +1915,13 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
                                     // need it for itself and don't let
                                     // ::IsDialogMessage() have it as it can
                                     // eat the Enter events sometimes
                                     // need it for itself and don't let
                                     // ::IsDialogMessage() have it as it can
                                     // eat the Enter events sometimes

-                                    return FALSE;
+                                    return false;

                                 }
                                 else if (!IsTopLevel())
                                 {
                                     // if not a top level window, let parent
                                     // handle it
                                 }
                                 else if (!IsTopLevel())
                                 {
                                     // if not a top level window, let parent
                                     // handle it

-                                    return FALSE;
+                                    return false;

                                 }
                                 //else: treat Enter as TAB: pass to the next
                                 //      control as this is the best thing to do
                                 }
                                 //else: treat Enter as TAB: pass to the next
                                 //      control as this is the best thing to do


@@ -1904,7 +1932,7 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
                     break;
 
                 default:
                     break;
 
                 default:

-                    bProcess = FALSE;
+                    bProcess = false;

             }
 
             if ( bProcess )
             }
 
             if ( bProcess )


@@ -1916,7 +1944,7 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
 
                 if ( GetEventHandler()->ProcessEvent(event) )
                 {
 
                 if ( GetEventHandler()->ProcessEvent(event) )
                 {

-                    return TRUE;
+                    return true;

                 }
             }
         }
                 }
             }
         }


@@ -1950,8 +1978,7 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
         }
 #endif // 1/0
 
         }
 #endif // 1/0
 

-        // we handle VK_ESCAPE ourselves in wxDialog::OnCharHook() and we
-        // shouldn't let IsDialogMessage() get it as it _always_ eats the
+        // don't let IsDialogMessage() get VK_ESCAPE as it _always_ eats the

         // message even when there is no cancel button and when the message is
         // needed by the control itself: in particular, it prevents the tree in
         // place edit control from being closed with Escape in a dialog
         // message even when there is no cancel button and when the message is
         // needed by the control itself: in particular, it prevents the tree in
         // place edit control from being closed with Escape in a dialog


@@ -1963,7 +1990,7 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
             // then
 
             // assume we can call it by default
             // then
 
             // assume we can call it by default

-            bool canSafelyCallIsDlgMsg = TRUE;
+            bool canSafelyCallIsDlgMsg = true;

 
             HWND hwndFocus = ::GetFocus();
 
 
             HWND hwndFocus = ::GetFocus();
 


@@ -1981,7 +2008,7 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
             if ( ::GetWindowLong(hwndFocus, GWL_EXSTYLE) & WS_EX_CONTROLPARENT )
             {
                 // passimistic by default
             if ( ::GetWindowLong(hwndFocus, GWL_EXSTYLE) & WS_EX_CONTROLPARENT )
             {
                 // passimistic by default

-                canSafelyCallIsDlgMsg = FALSE;
+                canSafelyCallIsDlgMsg = false;

                 for ( wxWindowList::compatibility_iterator node = GetChildren().GetFirst();
                       node;
                       node = node->GetNext() )
                 for ( wxWindowList::compatibility_iterator node = GetChildren().GetFirst();
                       node;
                       node = node->GetNext() )


@@ -1992,7 +2019,7 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
                                 WS_EX_CONTROLPARENT) )
                     {
                         // it shouldn't hang...
                                 WS_EX_CONTROLPARENT) )
                     {
                         // it shouldn't hang...

-                        canSafelyCallIsDlgMsg = TRUE;
+                        canSafelyCallIsDlgMsg = true;

 
                         break;
                     }
 
                         break;
                     }


@@ -2012,7 +2039,7 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
                             !::IsWindowVisible(hwndFocus) )
                     {
                         // it would enter an infinite loop if we do this!
                             !::IsWindowVisible(hwndFocus) )
                     {
                         // it would enter an infinite loop if we do this!

-                        canSafelyCallIsDlgMsg = FALSE;
+                        canSafelyCallIsDlgMsg = false;

 
                         break;
                     }
 
                         break;
                     }


@@ -2033,7 +2060,7 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
             if ( canSafelyCallIsDlgMsg && ::IsDialogMessage(GetHwnd(), msg) )
             {
                 // IsDialogMessage() did something...
             if ( canSafelyCallIsDlgMsg && ::IsDialogMessage(GetHwnd(), msg) )
             {
                 // IsDialogMessage() did something...

-                return TRUE;
+                return true;

             }
         }
     }
             }
         }
     }


@@ -2049,7 +2076,7 @@ bool wxWindowMSW::MSWProcessMessage(WXMSG* pMsg)
     }
 #endif // wxUSE_TOOLTIPS
 
     }
 #endif // wxUSE_TOOLTIPS
 

-    return FALSE;
+    return false;

 }
 
 bool wxWindowMSW::MSWTranslateMessage(WXMSG* pMsg)
 }
 
 bool wxWindowMSW::MSWTranslateMessage(WXMSG* pMsg)


@@ -2058,14 +2085,14 @@ bool wxWindowMSW::MSWTranslateMessage(WXMSG* pMsg)
     return m_acceleratorTable.Translate(this, pMsg);
 #else
     (void) pMsg;
     return m_acceleratorTable.Translate(this, pMsg);
 #else
     (void) pMsg;

-    return FALSE;
+    return false;

 #endif // wxUSE_ACCEL
 }
 
 bool wxWindowMSW::MSWShouldPreProcessMessage(WXMSG* WXUNUSED(pMsg))
 {
     // preprocess all messages by default
 #endif // wxUSE_ACCEL
 }
 
 bool wxWindowMSW::MSWShouldPreProcessMessage(WXMSG* WXUNUSED(pMsg))
 {
     // preprocess all messages by default

-    return TRUE;
+    return true;

 }
 
 // ---------------------------------------------------------------------------
 }
 
 // ---------------------------------------------------------------------------


@@ -2166,16 +2193,16 @@ LRESULT WXDLLEXPORT APIENTRY _EXPORT wxWndProc(HWND hWnd, UINT message, WPARAM w
     return rc;
 }
 
     return rc;
 }
 

-long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam)
+WXLRESULT wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam)

 {
     // did we process the message?
 {
     // did we process the message?

-    bool processed = FALSE;
+    bool processed = false;

 
     // the return value
     union
     {
         bool        allow;
 
     // the return value
     union
     {
         bool        allow;

-        long        result;
+        WXLRESULT   result;

         WXHICON     hIcon;
         WXHBRUSH    hBrush;
     } rc;
         WXHICON     hIcon;
         WXHBRUSH    hBrush;
     } rc;


@@ -2198,7 +2225,7 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
             break;
 
         case WM_DESTROY:
             break;
 
         case WM_DESTROY:

-            // never set processed to TRUE and *always* pass WM_DESTROY to
+            // never set processed to true and *always* pass WM_DESTROY to

             // DefWindowProc() as Windows may do some internal cleanup when
             // processing it and failing to pass the message along may cause
             // memory and resource leaks!
             // DefWindowProc() as Windows may do some internal cleanup when
             // processing it and failing to pass the message along may cause
             // memory and resource leaks!


@@ -2337,12 +2364,12 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
             // Universal uses its own wxFrame/wxDialog, so we don't receive
             // close events unless we have this.
             Close();
             // Universal uses its own wxFrame/wxDialog, so we don't receive
             // close events unless we have this.
             Close();

-            processed = TRUE;
+            processed = true;

             rc.result = TRUE;
 #else
             // don't let the DefWindowProc() destroy our window - we'll do it
             // ourselves in ~wxWindow
             rc.result = TRUE;
 #else
             // don't let the DefWindowProc() destroy our window - we'll do it
             // ourselves in ~wxWindow

-            processed = TRUE;
+            processed = true;

             rc.result = TRUE;
 #endif
             break;
             rc.result = TRUE;
 #endif
             break;


@@ -2371,7 +2398,7 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
             // up in the OnIdle code as before, for this special case )
             if ( /*IsComposite() && */ !IsMouseInWindow() )
             {
             // up in the OnIdle code as before, for this special case )
             if ( /*IsComposite() && */ !IsMouseInWindow() )
             {

-                m_mouseInWindow = FALSE;
+                m_mouseInWindow = false;

 
                 // Unfortunately no mouse state is passed with a WM_MOUSE_LEAVE 
                 int state = 0;
 
                 // Unfortunately no mouse state is passed with a WM_MOUSE_LEAVE 
                 int state = 0;


@@ -2438,7 +2465,7 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
                 {
                     if (!win->IsEnabled())
                     {
                 {
                     if (!win->IsEnabled())
                     {

-                        processed = TRUE;
+                        processed = true;

                         break;
                     }
 
                         break;
                     }
 


@@ -2520,7 +2547,7 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
             processed = HandleNotify((int)wParam, lParam, &rc.result);
             break;
 
             processed = HandleNotify((int)wParam, lParam, &rc.result);
             break;
 

-            // for these messages we must return TRUE if process the message
+            // for these messages we must return true if process the message

 #ifdef WM_DRAWITEM
         case WM_DRAWITEM:
         case WM_MEASUREITEM:
 #ifdef WM_DRAWITEM
         case WM_DRAWITEM:
         case WM_MEASUREITEM:


@@ -2557,7 +2584,7 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
                                  DLGC_WANTALLKEYS;
                 }
 
                                  DLGC_WANTALLKEYS;
                 }
 

-                processed = TRUE;
+                processed = true;

             }
             //else: get the dlg code from the DefWindowProc()
             break;
             }
             //else: get the dlg code from the DefWindowProc()
             break;


@@ -2569,7 +2596,7 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
             m_lastKeydownProcessed = HandleKeyDown((WORD) wParam, lParam);
             if ( m_lastKeydownProcessed )
             {
             m_lastKeydownProcessed = HandleKeyDown((WORD) wParam, lParam);
             if ( m_lastKeydownProcessed )
             {

-                processed = TRUE;
+                processed = true;

             }
 
             if ( !processed )
             }
 
             if ( !processed )


@@ -2584,7 +2611,7 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
                     case VK_CAPITAL:
                     case VK_NUMLOCK:
                     case VK_SCROLL:
                     case VK_CAPITAL:
                     case VK_NUMLOCK:
                     case VK_SCROLL:

-                        processed = TRUE;
+                        processed = true;

                         break;
 
                     // avoid duplicate messages to OnChar for these ASCII keys:
                         break;
 
                     // avoid duplicate messages to OnChar for these ASCII keys:


@@ -2610,10 +2637,10 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
                     case VK_OEM_COMMA:
                     case VK_OEM_MINUS:
                     case VK_OEM_PERIOD:
                     case VK_OEM_COMMA:
                     case VK_OEM_MINUS:
                     case VK_OEM_PERIOD:

-                        // but set processed to FALSE, not TRUE to still pass them
+                        // but set processed to false, not true to still pass them

                         // to the control's default window proc - otherwise
                         // built-in keyboard handling won't work
                         // to the control's default window proc - otherwise
                         // built-in keyboard handling won't work

-                        processed = FALSE;
+                        processed = false;

                         break;
 
 #ifdef VK_APPS
                         break;
 
 #ifdef VK_APPS


@@ -2636,7 +2663,7 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
                 }
             }
             if (message == WM_SYSKEYDOWN)  // Let Windows still handle the SYSKEYs
                 }
             }
             if (message == WM_SYSKEYDOWN)  // Let Windows still handle the SYSKEYs

-                processed = FALSE;
+                processed = false;

             break;
 
         case WM_SYSKEYUP:
             break;
 
         case WM_SYSKEYUP:


@@ -2665,12 +2692,12 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
                 // The key was handled in the EVT_KEY_DOWN and handling
                 // a key in an EVT_KEY_DOWN handler is meant, by
                 // design, to prevent EVT_CHARs from happening
                 // The key was handled in the EVT_KEY_DOWN and handling
                 // a key in an EVT_KEY_DOWN handler is meant, by
                 // design, to prevent EVT_CHARs from happening

-                m_lastKeydownProcessed = FALSE;
-                processed = TRUE;
+                m_lastKeydownProcessed = false;
+                processed = true;

             }
             else
             {
             }
             else
             {

-                processed = HandleChar((WORD)wParam, lParam, TRUE);
+                processed = HandleChar((WORD)wParam, lParam, true);

             }
             break;
 
             }
             break;
 


@@ -2821,7 +2848,7 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
                 {
 #endif
                     wxWindowMSW* subjectOfHelp = this;
                 {
 #endif
                     wxWindowMSW* subjectOfHelp = this;

-                    bool eventProcessed = FALSE;
+                    bool eventProcessed = false;

                     while (subjectOfHelp && !eventProcessed)
                     {
                         wxHelpEvent helpEvent(wxEVT_HELP,
                     while (subjectOfHelp && !eventProcessed)
                     {
                         wxHelpEvent helpEvent(wxEVT_HELP,


@@ -2852,7 +2879,7 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
                     processed = GetEventHandler()->ProcessEvent(helpEvent);
 
                 }
                     processed = GetEventHandler()->ProcessEvent(helpEvent);
 
                 }

-                //else: processed is already FALSE
+                //else: processed is already false

 #endif
             }
             break;
 #endif
             }
             break;


@@ -2866,8 +2893,20 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
                 wxPoint pt(GET_X_LPARAM(lParam), GET_Y_LPARAM(lParam));
 
                 wxContextMenuEvent evtCtx(wxEVT_CONTEXT_MENU, GetId(), pt);
                 wxPoint pt(GET_X_LPARAM(lParam), GET_Y_LPARAM(lParam));
 
                 wxContextMenuEvent evtCtx(wxEVT_CONTEXT_MENU, GetId(), pt);

-                evtCtx.SetEventObject(this);
-                processed = GetEventHandler()->ProcessEvent(evtCtx);
+
+                // we could have got an event from our child, reflect it back
+                // to it if this is the case
+                wxWindowMSW *win = NULL;
+                if ( (WXHWND)wParam != m_hWnd )
+                {
+                    win = FindItemByHWND((WXHWND)wParam);
+                }
+
+                if ( !win )
+                    win = this;
+
+                evtCtx.SetEventObject(win);
+                processed = win->GetEventHandler()->ProcessEvent(evtCtx);

             }
             break;
 #endif
             }
             break;
 #endif


@@ -2881,7 +2920,7 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
                 if ( i != wxNOT_FOUND )
                 {
                     rc.result = MAKELRESULT(i, MNC_EXECUTE);
                 if ( i != wxNOT_FOUND )
                 {
                     rc.result = MAKELRESULT(i, MNC_EXECUTE);

-                    processed = TRUE;
+                    processed = true;

                 }
             }
             break;
                 }
             }
             break;


@@ -2957,7 +2996,7 @@ bool wxWindowMSW::MSWGetCreateWindowCoords(const wxPoint& pos,
     static const int DEFAULT_W = 400;
     static const int DEFAULT_H = 250;
 
     static const int DEFAULT_W = 400;
     static const int DEFAULT_H = 250;
 

-    bool nonDefault = FALSE;
+    bool nonDefault = false;

 
     if ( pos.x == -1 )
     {
 
     if ( pos.x == -1 )
     {


@@ -2974,7 +3013,7 @@ bool wxWindowMSW::MSWGetCreateWindowCoords(const wxPoint& pos,
         x = pos.x;
         y = pos.y == -1 ? DEFAULT_Y : pos.y;
 
         x = pos.x;
         y = pos.y == -1 ? DEFAULT_Y : pos.y;
 

-        nonDefault = TRUE;
+        nonDefault = true;

     }
 
     /*
     }
 
     /*


@@ -3021,7 +3060,7 @@ bool wxWindowMSW::MSWGetCreateWindowCoords(const wxPoint& pos,
         w = size.x;
         h = size.y == -1 ? DEFAULT_H  : size.y;
 
         w = size.x;
         h = size.y == -1 ? DEFAULT_H  : size.y;
 

-        nonDefault = TRUE;
+        nonDefault = true;

     }
 
     AdjustForParentClientOrigin(x, y);
     }
 
     AdjustForParentClientOrigin(x, y);


@@ -3097,14 +3136,14 @@ bool wxWindowMSW::MSWCreate(const wxChar *wclass,
     {
         wxLogSysError(_("Can't create window of class %s"), wclass);
 
     {
         wxLogSysError(_("Can't create window of class %s"), wclass);
 

-        return FALSE;
+        return false;

     }
 
     SubclassWin(m_hWnd);
 
     SetFont(wxSystemSettings::GetFont(wxSYS_DEFAULT_GUI_FONT));
 
     }
 
     SubclassWin(m_hWnd);
 
     SetFont(wxSystemSettings::GetFont(wxSYS_DEFAULT_GUI_FONT));
 

-    return TRUE;
+    return true;

 }
 
 // ===========================================================================
 }
 
 // ===========================================================================


@@ -3144,7 +3183,7 @@ bool wxWindowMSW::HandleNotify(int idCtrl, WXLPARAM lParam, WXLPARAM *result)
         wxWindow *child = node->GetData();
         if ( child->MSWOnNotify(idCtrl, lParam, result) )
         {
         wxWindow *child = node->GetData();
         if ( child->MSWOnNotify(idCtrl, lParam, result) )
         {

-            return TRUE;
+            return true;

         }
 
         node = node->GetNext();
         }
 
         node = node->GetNext();


@@ -3154,7 +3193,7 @@ bool wxWindowMSW::HandleNotify(int idCtrl, WXLPARAM lParam, WXLPARAM *result)
     // by default, handle it ourselves
     return MSWOnNotify(idCtrl, lParam, result);
 #else // __WXMICROWIN__
     // by default, handle it ourselves
     return MSWOnNotify(idCtrl, lParam, result);
 #else // __WXMICROWIN__

-    return FALSE;
+    return false;

 #endif
 }
 
 #endif
 }
 


@@ -3170,59 +3209,71 @@ bool wxWindowMSW::HandleTooltipNotify(WXUINT code,
     // we need to handle it as well, otherwise no tooltips will be shown in
     // this case
 #ifndef __WXWINCE__
     // we need to handle it as well, otherwise no tooltips will be shown in
     // this case
 #ifndef __WXWINCE__

-    if ( !(code == (WXUINT) TTN_NEEDTEXTA || code == (WXUINT) TTN_NEEDTEXTW) || ttip.empty() )
+    if ( !(code == (WXUINT) TTN_NEEDTEXTA || code == (WXUINT) TTN_NEEDTEXTW)
+            || ttip.empty() )

     {
         // not a tooltip message or no tooltip to show anyhow
     {
         // not a tooltip message or no tooltip to show anyhow

-        return FALSE;
+        return false;

     }
 #endif
 
     LPTOOLTIPTEXT ttText = (LPTOOLTIPTEXT)lParam;
 
     }
 #endif
 
     LPTOOLTIPTEXT ttText = (LPTOOLTIPTEXT)lParam;
 

+    // We don't want to use the szText buffer because it has a limit of 80
+    // bytes and this is not enough, especially for Unicode build where it
+    // limits the tooltip string length to only 40 characters
+    //
+    // The best would be, of course, to not impose any length limitations at
+    // all but then the buffer would have to be dynamic and someone would have
+    // to free it and we don't have the tooltip owner object here any more, so
+    // for now use our own static buffer with a higher fixed max length.
+    //
+    // Note that using a static buffer should not be a problem as only a single
+    // tooltip can be shown at the same time anyhow.

 #if !wxUSE_UNICODE
 #if !wxUSE_UNICODE

-    if ( code == (WXUINT) TTN_NEEDTEXTA )
-    {
-        // we pass just the pointer as we store the string internally anyhow
-        ttText->lpszText = (char *)ttip.c_str();
-    }
-    else // TTN_NEEDTEXTW
-#endif // !Unicode
+    if ( code == (WXUINT) TTN_NEEDTEXTW )

     {
     {

-#if wxUSE_UNICODE
-        // in Unicode mode this is just what we need
-        ttText->lpszText = (wxChar *)ttip.c_str();
-#else // !Unicode
-/*
-        MultiByteToWideChar(CP_ACP, 0, ttip, ttip.length()+1,
-                            (wchar_t *)ttText->szText,
-                            sizeof(ttText->szText) / sizeof(wchar_t));
-*/
-        // Fix by dimitrishortcut: see patch 771772
-
-        // FIXME: szText has a max of 80 bytes, so limit the tooltip string
-        // length accordingly. Ideally lpszText should be used, but who
-        // would be responsible for freeing the buffer?
+        // We need to convert tooltip from multi byte to Unicode on the fly.
+        static wchar_t buf[513];

 
 

-        // Maximum length of a tip is 39 characters. 39 is 80/2 minus 1 byte
-        // needed for NULL character.
-        size_t tipLength = wxMin(ttip.Len(), 39);
+        // Truncate tooltip length if needed as otherwise we might not have
+        // enough space for it in the buffer and MultiByteToWideChar() would
+        // return an error
+        size_t tipLength = wxMin(ttip.Len(), WXSIZEOF(buf) - 1);

 
         // Convert to WideChar without adding the NULL character. The NULL
 
         // Convert to WideChar without adding the NULL character. The NULL

-        // character is added afterwards (Could have used ttip.Left(tipLength)
-        // and a cchMultiByte parameter of tipLength+1, but this is more
-        //efficient.
-        ::MultiByteToWideChar(CP_ACP, 0, ttip, tipLength,
-                             (wchar_t *)ttText->szText,
-                             sizeof(ttText->szText) / sizeof(wchar_t));
-
-        // Add the NULL character.
-        ttText->szText[tipLength*2+0] = '\0';
-        ttText->szText[tipLength*2+1] = '\0';
+        // character is added afterwards (this is more efficient).
+        int len = ::MultiByteToWideChar
+                    (
+                        CP_ACP,
+                        0,                      // no flags
+                        ttip,
+                        tipLength,
+                        buf,
+                        WXSIZEOF(buf) - 1
+                    );
+
+        if ( !len )
+        {
+            wxLogLastError(_T("MultiByteToWideChar()"));
+        }

 
 

-#endif // Unicode/!Unicode
+        buf[len] = L'\0';
+        ttText->lpszText = (LPSTR) buf;
+    }
+    else // TTN_NEEDTEXTA
+#endif // !wxUSE_UNICODE
+    {
+        // we get here if we got TTN_NEEDTEXTA (only happens in ANSI build) or
+        // if we got TTN_NEEDTEXTW in Unicode build: in this case we just have
+        // to copy the string we have into the buffer
+        static wxChar buf[513];
+        wxStrncpy(buf, ttip.c_str(), WXSIZEOF(buf) - 1);
+        buf[WXSIZEOF(buf) - 1] = _T('\0');
+        ttText->lpszText = buf;

     }
 
     }
 

-    return TRUE;
+    return true;

 }
 
 #endif // wxUSE_TOOLTIPS
 }
 
 #endif // wxUSE_TOOLTIPS


@@ -3238,12 +3289,12 @@ bool wxWindowMSW::MSWOnNotify(int WXUNUSED(idCtrl),
         if ( HandleTooltipNotify(hdr->code, lParam, m_tooltip->GetTip()))
         {
             // processed
         if ( HandleTooltipNotify(hdr->code, lParam, m_tooltip->GetTip()))
         {
             // processed

-            return TRUE;
+            return true;

         }
     }
 #endif // wxUSE_TOOLTIPS
 
         }
     }
 #endif // wxUSE_TOOLTIPS
 

-    return FALSE;
+    return false;

 }
 
 #endif // __WIN95__
 }
 
 #endif // __WIN95__


@@ -3257,7 +3308,7 @@ bool wxWindowMSW::HandleQueryEndSession(long logOff, bool *mayEnd)
 #ifndef __WXWINCE__
     wxCloseEvent event(wxEVT_QUERY_END_SESSION, -1);
     event.SetEventObject(wxTheApp);
 #ifndef __WXWINCE__
     wxCloseEvent event(wxEVT_QUERY_END_SESSION, -1);
     event.SetEventObject(wxTheApp);

-    event.SetCanVeto(TRUE);
+    event.SetCanVeto(true);

     event.SetLoggingOff(logOff == (long)ENDSESSION_LOGOFF);
 
     bool rc = wxTheApp->ProcessEvent(event);
     event.SetLoggingOff(logOff == (long)ENDSESSION_LOGOFF);
 
     bool rc = wxTheApp->ProcessEvent(event);


@@ -3271,7 +3322,7 @@ bool wxWindowMSW::HandleQueryEndSession(long logOff, bool *mayEnd)
 
     return rc;
 #else
 
     return rc;
 #else

-    return FALSE;
+    return false;

 #endif
 }
 
 #endif
 }
 


@@ -3280,20 +3331,20 @@ bool wxWindowMSW::HandleEndSession(bool endSession, long logOff)
 #ifndef __WXWINCE__
     // do nothing if the session isn't ending
     if ( !endSession )
 #ifndef __WXWINCE__
     // do nothing if the session isn't ending
     if ( !endSession )

-        return FALSE;
+        return false;

 
     // only send once
     if ( (this != wxTheApp->GetTopWindow()) )
 
     // only send once
     if ( (this != wxTheApp->GetTopWindow()) )

-        return FALSE;
+        return false;

 
     wxCloseEvent event(wxEVT_END_SESSION, -1);
     event.SetEventObject(wxTheApp);
 
     wxCloseEvent event(wxEVT_END_SESSION, -1);
     event.SetEventObject(wxTheApp);

-    event.SetCanVeto(FALSE);
+    event.SetCanVeto(false);

     event.SetLoggingOff( (logOff == (long)ENDSESSION_LOGOFF) );
 
     return wxTheApp->ProcessEvent(event);
 #else
     event.SetLoggingOff( (logOff == (long)ENDSESSION_LOGOFF) );
 
     return wxTheApp->ProcessEvent(event);
 #else

-    return FALSE;
+    return false;

 #endif
 }
 
 #endif
 }
 


@@ -3315,9 +3366,9 @@ bool wxWindowMSW::HandleCreate(WXLPCREATESTRUCT cs, bool *mayCreate)
     wxWindowCreateEvent event((wxWindow *)this);
     (void)GetEventHandler()->ProcessEvent(event);
 
     wxWindowCreateEvent event((wxWindow *)this);
     (void)GetEventHandler()->ProcessEvent(event);
 

-    *mayCreate = TRUE;
+    *mayCreate = true;

 
 

-    return TRUE;
+    return true;

 }
 
 bool wxWindowMSW::HandleDestroy()
 }
 
 bool wxWindowMSW::HandleDestroy()


@@ -3336,7 +3387,7 @@ bool wxWindowMSW::HandleDestroy()
 #endif // wxUSE_DRAG_AND_DROP
 
     // WM_DESTROY handled
 #endif // wxUSE_DRAG_AND_DROP
 
     // WM_DESTROY handled

-    return TRUE;
+    return true;

 }
 
 // ---------------------------------------------------------------------------
 }
 
 // ---------------------------------------------------------------------------


@@ -3375,7 +3426,7 @@ bool wxWindowMSW::HandleSetFocus(WXHWND hwnd)
     // after the control gets to process it from EN_FOCUS handler
     if ( wxDynamicCastThis(wxTextCtrl) )
     {
     // after the control gets to process it from EN_FOCUS handler
     if ( wxDynamicCastThis(wxTextCtrl) )
     {

-        return FALSE;
+        return false;

     }
 #endif // wxUSE_TEXTCTRL
 
     }
 #endif // wxUSE_TEXTCTRL
 


@@ -3404,7 +3455,7 @@ bool wxWindowMSW::HandleKillFocus(WXHWND hwnd)
     wxTextCtrl *ctrl = wxDynamicCastThis(wxTextCtrl);
     if ( ctrl )
     {
     wxTextCtrl *ctrl = wxDynamicCastThis(wxTextCtrl);
     if ( ctrl )
     {

-        return FALSE;
+        return false;

     }
 #endif
 
     }
 #endif
 


@@ -3412,7 +3463,7 @@ bool wxWindowMSW::HandleKillFocus(WXHWND hwnd)
     // only cause problems if the event handler tries to access the object.
     if ( m_isBeingDeleted )
     {
     // only cause problems if the event handler tries to access the object.
     if ( m_isBeingDeleted )
     {

-        return FALSE;
+        return false;

     }
 
     wxFocusEvent event(wxEVT_KILL_FOCUS, m_windowId);
     }
 
     wxFocusEvent event(wxEVT_KILL_FOCUS, m_windowId);


@@ -3447,7 +3498,7 @@ bool wxWindowMSW::HandleInitDialog(WXHWND WXUNUSED(hWndFocus))
 bool wxWindowMSW::HandleDropFiles(WXWPARAM wParam)
 {
 #if defined (__WXMICROWIN__) || defined(__WXWINCE__)
 bool wxWindowMSW::HandleDropFiles(WXWPARAM wParam)
 {
 #if defined (__WXMICROWIN__) || defined(__WXWINCE__)

-    return FALSE;
+    return false;

 #else // __WXMICROWIN__
     HDROP hFilesInfo = (HDROP) wParam;
 
 #else // __WXMICROWIN__
     HDROP hFilesInfo = (HDROP) wParam;
 


@@ -3500,7 +3551,7 @@ bool wxWindowMSW::HandleSetCursor(WXHWND WXUNUSED(hWnd),
 
     if ( nHitTest != HTCLIENT )
     {
 
     if ( nHitTest != HTCLIENT )
     {

-        return FALSE;
+        return false;

     }
 
     HCURSOR hcursor = 0;
     }
 
     HCURSOR hcursor = 0;


@@ -3561,12 +3612,12 @@ bool wxWindowMSW::HandleSetCursor(WXHWND WXUNUSED(hWnd),
         ::SetCursor(hcursor);
 
         // cursor set, stop here
         ::SetCursor(hcursor);
 
         // cursor set, stop here

-        return TRUE;
+        return true;

     }
 #endif // __WXMICROWIN__
 
     // pass up the window chain
     }
 #endif // __WXMICROWIN__
 
     // pass up the window chain

-    return FALSE;
+    return false;

 }
 
 // ---------------------------------------------------------------------------
 }
 
 // ---------------------------------------------------------------------------


@@ -3593,7 +3644,7 @@ wxWindowMSW::MSWOnDrawItem(int WXUNUSED_UNLESS_ODRAWN(id),
     {
         wxMenuItem *pMenuItem = (wxMenuItem *)(pDrawStruct->itemData);
 
     {
         wxMenuItem *pMenuItem = (wxMenuItem *)(pDrawStruct->itemData);
 

-        wxCHECK( pMenuItem->IsKindOf(CLASSINFO(wxMenuItem)), FALSE );
+        wxCHECK( pMenuItem->IsKindOf(CLASSINFO(wxMenuItem)), false );

 
         // prepare to call OnDrawItem(): notice using of wxDCTemp to prevent
         // the DC from being released
 
         // prepare to call OnDrawItem(): notice using of wxDCTemp to prevent
         // the DC from being released


@@ -3631,7 +3682,7 @@ wxWindowMSW::MSWOnDrawItem(int WXUNUSED_UNLESS_ODRAWN(id),
 
 #endif // wxUSE_CONTROLS
 
 
 #endif // wxUSE_CONTROLS
 

-    return FALSE;
+    return false;

 }
 
 bool
 }
 
 bool


@@ -3646,10 +3697,15 @@ wxWindowMSW::MSWOnMeasureItem(int WXUNUSED_UNLESS_ODRAWN(id),
     {
         wxMenuItem *pMenuItem = (wxMenuItem *)(pMeasureStruct->itemData);
 
     {
         wxMenuItem *pMenuItem = (wxMenuItem *)(pMeasureStruct->itemData);
 

-        wxCHECK( pMenuItem->IsKindOf(CLASSINFO(wxMenuItem)), FALSE );
+        wxCHECK( pMenuItem->IsKindOf(CLASSINFO(wxMenuItem)), false );
+
+        size_t w, h;
+        bool rc = pMenuItem->OnMeasureItem(&w, &h);

 
 

-        return pMenuItem->OnMeasureItem(&pMeasureStruct->itemWidth,
-                                        &pMeasureStruct->itemHeight);
+        pMeasureStruct->itemWidth = w;
+        pMeasureStruct->itemHeight = h;
+
+        return rc;

     }
 
     wxControl *item = wxDynamicCast(FindItem(id), wxControl);
     }
 
     wxControl *item = wxDynamicCast(FindItem(id), wxControl);


@@ -3659,7 +3715,7 @@ wxWindowMSW::MSWOnMeasureItem(int WXUNUSED_UNLESS_ODRAWN(id),
     }
 #endif // wxUSE_OWNER_DRAWN
 
     }
 #endif // wxUSE_OWNER_DRAWN
 

-    return FALSE;
+    return false;

 }
 
 // ---------------------------------------------------------------------------
 }
 
 // ---------------------------------------------------------------------------


@@ -3675,7 +3731,7 @@ bool wxWindowMSW::HandleSysColorChange()
 
     // always let the system carry on the default processing to allow the
     // native controls to react to the colours update
 
     // always let the system carry on the default processing to allow the
     // native controls to react to the colours update

-    return FALSE;
+    return false;

 }
 
 bool wxWindowMSW::HandleDisplayChange()
 }
 
 bool wxWindowMSW::HandleDisplayChange()


@@ -3698,7 +3754,7 @@ bool wxWindowMSW::HandleCtlColor(WXHBRUSH *brush,
     WXHBRUSH hBrush = 0;
 
 #ifdef __WXWINCE__
     WXHBRUSH hBrush = 0;
 
 #ifdef __WXWINCE__

-    if (FALSE)
+    if (false)

 #else
     if ( nCtlColor == CTLCOLOR_DLG )
 #endif
 #else
     if ( nCtlColor == CTLCOLOR_DLG )
 #endif


@@ -3708,7 +3764,7 @@ bool wxWindowMSW::HandleCtlColor(WXHBRUSH *brush,
 #if wxUSE_CONTROLS
     else
     {
 #if wxUSE_CONTROLS
     else
     {

-        wxControl *item = (wxControl *)FindItemByHWND(pWnd, TRUE);
+        wxControl *item = (wxControl *)FindItemByHWND(pWnd, true);

         if ( item )
             hBrush = item->OnCtlColor(pDC, pWnd, nCtlColor, message, wParam, lParam);
     }
         if ( item )
             hBrush = item->OnCtlColor(pDC, pWnd, nCtlColor, message, wParam, lParam);
     }


@@ -3719,7 +3775,7 @@ bool wxWindowMSW::HandleCtlColor(WXHBRUSH *brush,
 
     return hBrush != 0;
 #else // __WXMICROWIN__
 
     return hBrush != 0;
 #else // __WXMICROWIN__

-    return FALSE;
+    return false;

 #endif
 }
 
 #endif
 }
 


@@ -3764,7 +3820,7 @@ bool wxWindowMSW::HandlePaletteChanged(WXHWND hWndPalChange)
 
             // now check for the need to redraw
             if (result > 0)
 
             // now check for the need to redraw
             if (result > 0)

-                InvalidateRect((HWND) hWndPalChange, NULL, TRUE);
+                ::InvalidateRect((HWND) hWndPalChange, NULL, TRUE);

         }
 
     }
         }
 
     }


@@ -3825,7 +3881,7 @@ void wxWindowMSW::OnSysColourChanged(wxSysColourChangedEvent& WXUNUSED(event))
     if ( IsTopLevel() )
     {
         // FIXME-MT
     if ( IsTopLevel() )
     {
         // FIXME-MT

-        gs_hasStdCmap = FALSE;
+        gs_hasStdCmap = false;

     }
     wxWindowList::compatibility_iterator node = GetChildren().GetFirst();
     while ( node )
     }
     wxWindowList::compatibility_iterator node = GetChildren().GetFirst();
     while ( node )


@@ -3866,7 +3922,7 @@ extern wxCOLORMAP *wxGetStdColourMap()
 
     if ( !gs_hasStdCmap )
     {
 
     if ( !gs_hasStdCmap )
     {

-        static bool s_coloursInit = FALSE;
+        static bool s_coloursInit = false;

 
         if ( !s_coloursInit )
         {
 
         if ( !s_coloursInit )
         {


@@ -3904,10 +3960,10 @@ extern wxCOLORMAP *wxGetStdColourMap()
                 //s_stdColours[5] = RGB(255,000,255);     // magenta
             }
 
                 //s_stdColours[5] = RGB(255,000,255);     // magenta
             }
 

-            s_coloursInit = TRUE;
+            s_coloursInit = true;

         }
 
         }
 

-        gs_hasStdCmap = TRUE;
+        gs_hasStdCmap = true;

 
         // create the colour map
 #define INIT_CMAP_ENTRY(col) \
 
         // create the colour map
 #define INIT_CMAP_ENTRY(col) \


@@ -3975,12 +4031,12 @@ bool wxWindowMSW::HandleEraseBkgnd(WXHDC hdc)
 {
     // Prevents flicker when dragging
     if ( ::IsIconic(GetHwnd()) )
 {
     // Prevents flicker when dragging
     if ( ::IsIconic(GetHwnd()) )

-        return TRUE;
+        return true;

 
 #if 0
     if (GetParent() && GetParent()->GetExtraStyle() & wxWS_EX_THEMED_BACKGROUND)
     {
 
 #if 0
     if (GetParent() && GetParent()->GetExtraStyle() & wxWS_EX_THEMED_BACKGROUND)
     {

-        return FALSE;
+        return false;

     }
 
     if (GetExtraStyle() & wxWS_EX_THEMED_BACKGROUND)
     }
 
     if (GetExtraStyle() & wxWS_EX_THEMED_BACKGROUND)


@@ -3994,7 +4050,7 @@ bool wxWindowMSW::HandleEraseBkgnd(WXHDC hdc)
                 ::GetClientRect((HWND) GetHWND(), (RECT*) & rect);
                 wxUxThemeEngine::Get()->m_pfnDrawThemeBackground(hTheme, hdc, 10 /* TABP_BODY */, 0, &rect, &rect);
                 wxUxThemeEngine::Get()->m_pfnCloseThemeData(hTheme);
                 ::GetClientRect((HWND) GetHWND(), (RECT*) & rect);
                 wxUxThemeEngine::Get()->m_pfnDrawThemeBackground(hTheme, hdc, 10 /* TABP_BODY */, 0, &rect, &rect);
                 wxUxThemeEngine::Get()->m_pfnCloseThemeData(hTheme);

-                return TRUE;
+                return true;

             }
         }
     }
             }
         }
     }


@@ -4109,11 +4165,11 @@ bool wxWindowMSW::HandleSizing(wxRect& rect)
 bool wxWindowMSW::HandleGetMinMaxInfo(void *mmInfo)
 {
 #ifdef __WXWINCE__
 bool wxWindowMSW::HandleGetMinMaxInfo(void *mmInfo)
 {
 #ifdef __WXWINCE__

-    return FALSE;
+    return false;

 #else
     MINMAXINFO *info = (MINMAXINFO *)mmInfo;
 
 #else
     MINMAXINFO *info = (MINMAXINFO *)mmInfo;
 

-    bool rc = FALSE;
+    bool rc = false;

 
     int minWidth = GetMinWidth(),
         minHeight = GetMinHeight(),
 
     int minWidth = GetMinWidth(),
         minHeight = GetMinHeight(),


@@ -4123,25 +4179,25 @@ bool wxWindowMSW::HandleGetMinMaxInfo(void *mmInfo)
     if ( minWidth != -1 )
     {
         info->ptMinTrackSize.x = minWidth;
     if ( minWidth != -1 )
     {
         info->ptMinTrackSize.x = minWidth;

-        rc = TRUE;
+        rc = true;

     }
 
     if ( minHeight != -1 )
     {
         info->ptMinTrackSize.y = minHeight;
     }
 
     if ( minHeight != -1 )
     {
         info->ptMinTrackSize.y = minHeight;

-        rc = TRUE;
+        rc = true;

     }
 
     if ( maxWidth != -1 )
     {
         info->ptMaxTrackSize.x = maxWidth;
     }
 
     if ( maxWidth != -1 )
     {
         info->ptMaxTrackSize.x = maxWidth;

-        rc = TRUE;
+        rc = true;

     }
 
     if ( maxHeight != -1 )
     {
         info->ptMaxTrackSize.y = maxHeight;
     }
 
     if ( maxHeight != -1 )
     {
         info->ptMaxTrackSize.y = maxHeight;

-        rc = TRUE;
+        rc = true;

     }
 
     return rc;
     }
 
     return rc;


@@ -4209,11 +4265,11 @@ bool wxWindowMSW::HandleCommand(WXWORD id, WXWORD cmd, WXHWND control)
         // wxSpinCtrl
         wxSpinCtrl *spin = wxSpinCtrl::GetSpinForTextCtrl(control);
         if ( spin && spin->ProcessTextCommand(cmd, id) )
         // wxSpinCtrl
         wxSpinCtrl *spin = wxSpinCtrl::GetSpinForTextCtrl(control);
         if ( spin && spin->ProcessTextCommand(cmd, id) )

-            return TRUE;
+            return true;

     }
 #endif // wxUSE_SPINCTRL
 
     }
 #endif // wxUSE_SPINCTRL
 

-    return FALSE;
+    return false;

 }
 
 bool wxWindowMSW::HandleSysCommand(WXWPARAM wParam, WXLPARAM WXUNUSED(lParam))
 }
 
 bool wxWindowMSW::HandleSysCommand(WXWPARAM wParam, WXLPARAM WXUNUSED(lParam))


@@ -4230,7 +4286,7 @@ bool wxWindowMSW::HandleSysCommand(WXWPARAM wParam, WXLPARAM WXUNUSED(lParam))
     }
 #endif
 
     }
 #endif
 

-    return FALSE;
+    return false;

 }
 
 // ---------------------------------------------------------------------------
 }
 
 // ---------------------------------------------------------------------------


@@ -4371,7 +4427,7 @@ bool wxWindowMSW::HandleMouseMove(int x, int y, WXUINT flags)
         if ( !HasCapture() || IsMouseInWindow() )
         {
             // Generate an ENTER event
         if ( !HasCapture() || IsMouseInWindow() )
         {
             // Generate an ENTER event

-            m_mouseInWindow = TRUE;
+            m_mouseInWindow = true;

 #if _WIN32_WINNT >= 0x0400
 #ifndef __WXWINCE__
             TRACKMOUSEEVENT trackinfo;
 #if _WIN32_WINNT >= 0x0400
 #ifndef __WXWINCE__
             TRACKMOUSEEVENT trackinfo;


@@ -4403,7 +4459,7 @@ bool wxWindowMSW::HandleMouseMove(int x, int y, WXUINT flags)
     {
         m_lastMouseEvent = wxEVT_MOTION;
 
     {
         m_lastMouseEvent = wxEVT_MOTION;
 

-        return FALSE;
+        return false;

     }
 #endif // wxUSE_MOUSEEVENT_HACK
 
     }
 #endif // wxUSE_MOUSEEVENT_HACK
 


@@ -4443,7 +4499,7 @@ bool wxWindowMSW::HandleMouseWheel(WXWPARAM wParam, WXLPARAM lParam)
     (void) wParam;
     (void) lParam;
 
     (void) wParam;
     (void) lParam;
 

-    return FALSE;
+    return false;

 #endif
 }
 
 #endif
 }
 


@@ -4487,7 +4543,7 @@ wxKeyEvent wxWindowMSW::CreateKeyEvent(wxEventType evType,
     return event;
 }
 
     return event;
 }
 

-// isASCII is TRUE only when we're called from WM_CHAR handler and not from
+// isASCII is true only when we're called from WM_CHAR handler and not from

 // WM_KEYDOWN one
 bool wxWindowMSW::HandleChar(WXWPARAM wParam, WXLPARAM lParam, bool isASCII)
 {
 // WM_KEYDOWN one
 bool wxWindowMSW::HandleChar(WXWPARAM wParam, WXLPARAM lParam, bool isASCII)
 {


@@ -4496,7 +4552,7 @@ bool wxWindowMSW::HandleChar(WXWPARAM wParam, WXLPARAM lParam, bool isASCII)
     {
         // If 1 -> 26, translate to either special keycode or just set
         // ctrlDown.  IOW, Ctrl-C should result in keycode == 3 and
     {
         // If 1 -> 26, translate to either special keycode or just set
         // ctrlDown.  IOW, Ctrl-C should result in keycode == 3 and

-        // ControlDown() == TRUE.
+        // ControlDown() == true.

         id = wParam;
         if ( (id > 0) && (id < 27) )
         {
         id = wParam;
         if ( (id > 0) && (id < 27) )
         {


@@ -4515,7 +4571,7 @@ bool wxWindowMSW::HandleChar(WXWPARAM wParam, WXLPARAM lParam, bool isASCII)
                     break;
 
                 default:
                     break;
 
                 default:

-                    //ctrlDown = TRUE;
+                    //ctrlDown = true;

                     break;
             }
         }
                     break;
             }
         }


@@ -4526,8 +4582,8 @@ bool wxWindowMSW::HandleChar(WXWPARAM wParam, WXLPARAM lParam, bool isASCII)
         if ( id == 0 )
         {
             // it's ASCII and will be processed here only when called from
         if ( id == 0 )
         {
             // it's ASCII and will be processed here only when called from

-            // WM_CHAR (i.e. when isASCII = TRUE), don't process it now
-            return FALSE;
+            // WM_CHAR (i.e. when isASCII = true), don't process it now
+            return false;

         }
     }
 
         }
     }
 


@@ -4543,7 +4599,7 @@ bool wxWindowMSW::HandleChar(WXWPARAM wParam, WXLPARAM lParam, bool isASCII)
             (id >= 32 && id < 256) )
     {
         event.m_controlDown =
             (id >= 32 && id < 256) )
     {
         event.m_controlDown =

-        event.m_altDown = FALSE;
+        event.m_altDown = false;

     }
 
     return GetEventHandler()->ProcessEvent(event);
     }
 
     return GetEventHandler()->ProcessEvent(event);


@@ -4564,11 +4620,11 @@ bool wxWindowMSW::HandleKeyDown(WXWPARAM wParam, WXLPARAM lParam)
         wxKeyEvent event(CreateKeyEvent(wxEVT_KEY_DOWN, id, lParam, wParam));
         if ( GetEventHandler()->ProcessEvent(event) )
         {
         wxKeyEvent event(CreateKeyEvent(wxEVT_KEY_DOWN, id, lParam, wParam));
         if ( GetEventHandler()->ProcessEvent(event) )
         {

-            return TRUE;
+            return true;

         }
     }
 
         }
     }
 

-    return FALSE;
+    return false;

 }
 
 bool wxWindowMSW::HandleKeyUp(WXWPARAM wParam, WXLPARAM lParam)
 }
 
 bool wxWindowMSW::HandleKeyUp(WXWPARAM wParam, WXLPARAM lParam)


@@ -4585,10 +4641,10 @@ bool wxWindowMSW::HandleKeyUp(WXWPARAM wParam, WXLPARAM lParam)
     {
         wxKeyEvent event(CreateKeyEvent(wxEVT_KEY_UP, id, lParam, wParam));
         if ( GetEventHandler()->ProcessEvent(event) )
     {
         wxKeyEvent event(CreateKeyEvent(wxEVT_KEY_UP, id, lParam, wParam));
         if ( GetEventHandler()->ProcessEvent(event) )

-            return TRUE;
+            return true;

     }
 
     }
 

-    return FALSE;
+    return false;

 }
 
 int wxWindowMSW::HandleMenuChar(int chAccel, WXLPARAM lParam)
 }
 
 int wxWindowMSW::HandleMenuChar(int chAccel, WXLPARAM lParam)


@@ -4728,7 +4784,7 @@ bool wxWindowMSW::HandleJoystickEvent(WXUINT msg, int x, int y, WXUINT flags)
         default:
             wxFAIL_MSG(wxT("no such joystick event"));
 
         default:
             wxFAIL_MSG(wxT("no such joystick event"));
 

-            return FALSE;
+            return false;

     }
 
     wxJoystickEvent event(eventType, buttons, joystick, change);
     }
 
     wxJoystickEvent event(eventType, buttons, joystick, change);


@@ -4737,7 +4793,7 @@ bool wxWindowMSW::HandleJoystickEvent(WXUINT msg, int x, int y, WXUINT flags)
 
     return GetEventHandler()->ProcessEvent(event);
 #else
 
     return GetEventHandler()->ProcessEvent(event);
 #else

-    return FALSE;
+    return false;

 #endif
 }
 
 #endif
 }
 


@@ -4747,8 +4803,8 @@ bool wxWindowMSW::HandleJoystickEvent(WXUINT msg, int x, int y, WXUINT flags)
 
 bool wxWindowMSW::MSWOnScroll(int orientation, WXWORD wParam,
                               WXWORD pos, WXHWND control)
 
 bool wxWindowMSW::MSWOnScroll(int orientation, WXWORD wParam,
                               WXWORD pos, WXHWND control)

-{
-    if ( control )
+{    
+    if ( control && control != m_hWnd ) // Prevent infinite recursion

     {
         wxWindow *child = wxFindWinFromHandle(control);
         if ( child )
     {
         wxWindow *child = wxFindWinFromHandle(control);
         if ( child )


@@ -4815,7 +4871,7 @@ bool wxWindowMSW::MSWOnScroll(int orientation, WXWORD wParam,
         break;
 
     default:
         break;
 
     default:

-        return FALSE;
+        return false;

     }
 
     return GetEventHandler()->ProcessEvent(event);
     }
 
     return GetEventHandler()->ProcessEvent(event);


@@ -4957,7 +5013,7 @@ int wxCharCodeMSWToWX(int keySym)
 
 int wxCharCodeWXToMSW(int id, bool *isVirtual)
 {
 
 int wxCharCodeWXToMSW(int id, bool *isVirtual)
 {

-    *isVirtual = TRUE;
+    *isVirtual = true;

     int keySym;
     switch (id)
     {
     int keySym;
     switch (id)
     {


@@ -5024,7 +5080,7 @@ int wxCharCodeWXToMSW(int id, bool *isVirtual)
     case WXK_SCROLL:    keySym = VK_SCROLL; break;
     default:
         {
     case WXK_SCROLL:    keySym = VK_SCROLL; break;
     default:
         {

-            *isVirtual = FALSE;
+            *isVirtual = false;

             keySym = id;
             break;
         }
             keySym = id;
             break;
         }


@@ -5073,7 +5129,7 @@ extern wxWindow *wxGetWindowFromHWND(WXHWND hWnd)
             // do it as well, win would be already non NULL
             if ( ::SendMessage(hwnd, WM_GETDLGCODE, 0, 0) & DLGC_RADIOBUTTON )
             {
             // do it as well, win would be already non NULL
             if ( ::SendMessage(hwnd, WM_GETDLGCODE, 0, 0) & DLGC_RADIOBUTTON )
             {

-                win = (wxWindow *)::GetWindowLong(hwnd, GWL_USERDATA);
+                win = (wxWindow *)wxGetWindowUserData(hwnd);

             }
             //else: it's a wxRadioButton, not a radiobutton from wxRadioBox
 #endif // wxUSE_RADIOBOX
             }
             //else: it's a wxRadioButton, not a radiobutton from wxRadioBox
 #endif // wxUSE_RADIOBOX


@@ -5151,7 +5207,7 @@ wxKeyboardHook(int nCode, WORD wParam, DWORD lParam)
         {
             wxKeyEvent event(wxEVT_CHAR_HOOK);
             if ( (HIWORD(lParam) & KF_ALTDOWN) == KF_ALTDOWN )
         {
             wxKeyEvent event(wxEVT_CHAR_HOOK);
             if ( (HIWORD(lParam) & KF_ALTDOWN) == KF_ALTDOWN )

-                event.m_altDown = TRUE;
+                event.m_altDown = true;

 
             event.m_eventObject = NULL;
             event.m_keyCode = id;
 
             event.m_eventObject = NULL;
             event.m_keyCode = id;


@@ -5704,10 +5760,10 @@ bool wxWindowMSW::RegisterHotKey(int hotkeyId, int modifiers, int keycode)
     {
         wxLogLastError(_T("RegisterHotKey"));
 
     {
         wxLogLastError(_T("RegisterHotKey"));
 

-        return FALSE;
+        return false;

     }
 
     }
 

-    return TRUE;
+    return true;

 }
 
 bool wxWindowMSW::UnregisterHotKey(int hotkeyId)
 }
 
 bool wxWindowMSW::UnregisterHotKey(int hotkeyId)


@@ -5716,10 +5772,10 @@ bool wxWindowMSW::UnregisterHotKey(int hotkeyId)
     {
         wxLogLastError(_T("UnregisterHotKey"));
 
     {
         wxLogLastError(_T("UnregisterHotKey"));
 

-        return FALSE;
+        return false;

     }
 
     }
 

-    return TRUE;
+    return true;

 }
 
 bool wxWindowMSW::HandleHotKey(WXWPARAM wParam, WXLPARAM lParam)
 }
 
 bool wxWindowMSW::HandleHotKey(WXWPARAM wParam, WXLPARAM lParam)