Crash fix under VC++

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

diff --git a/src/x11/window.cpp b/src/x11/window.cpp
index cf10d5730f57893cab180d493dce9ff6012005a9..675d1a8b2fc1e17e6c3f4e6d2787fdab9f48672d 100644 (file)
--- a/src/x11/window.cpp
+++ b/src/x11/window.cpp
@@ -17,7 +17,7 @@
 // headers
 // ----------------------------------------------------------------------------
 
 // headers
 // ----------------------------------------------------------------------------
 

-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)

     #pragma implementation "window.h"
 #endif
 
     #pragma implementation "window.h"
 #endif
 


@@ -62,8 +62,6 @@
 // global variables for this module
 // ----------------------------------------------------------------------------
 
 // global variables for this module
 // ----------------------------------------------------------------------------
 

-extern wxHashTable *wxWidgetHashTable;
-extern wxHashTable *wxClientWidgetHashTable;

 static wxWindow* g_captureWindow = NULL;
 static GC g_eraseGC;
 
 static wxWindow* g_captureWindow = NULL;
 static GC g_eraseGC;
 


@@ -99,19 +97,15 @@ END_EVENT_TABLE()
 
 void wxWindowX11::Init()
 {
 
 void wxWindowX11::Init()
 {

-    // generic initializations first
-    InitBase();
-

     // X11-specific
     m_mainWindow = (WXWindow) 0;
     m_clientWindow = (WXWindow) 0;
     m_insertIntoMain = FALSE;
     m_updateNcArea = FALSE;
     // X11-specific
     m_mainWindow = (WXWindow) 0;
     m_clientWindow = (WXWindow) 0;
     m_insertIntoMain = FALSE;
     m_updateNcArea = FALSE;

-    
+

     m_winCaptured = FALSE;
     m_needsInputFocus = FALSE;
     m_isShown = TRUE;
     m_winCaptured = FALSE;
     m_needsInputFocus = FALSE;
     m_isShown = TRUE;

-    m_isBeingDeleted = FALSE;

     m_lastTS = 0;
     m_lastButton = 0;
 }
     m_lastTS = 0;
     m_lastButton = 0;
 }


@@ -123,7 +117,7 @@ bool wxWindowX11::Create(wxWindow *parent, wxWindowID id,
                       long style,
                       const wxString& name)
 {
                       long style,
                       const wxString& name)
 {

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

 
     CreateBase(parent, id, pos, size, style, wxDefaultValidator, name);
 
 
     CreateBase(parent, id, pos, size, style, wxDefaultValidator, name);
 


@@ -135,20 +129,20 @@ bool wxWindowX11::Create(wxWindow *parent, wxWindowID id,
     Colormap cm = DefaultColormap( xdisplay, xscreen );
 
     m_backgroundColour = wxSystemSettings::GetColour(wxSYS_COLOUR_3DFACE);
     Colormap cm = DefaultColormap( xdisplay, xscreen );
 
     m_backgroundColour = wxSystemSettings::GetColour(wxSYS_COLOUR_3DFACE);

-    m_backgroundColour.CalcPixel( (WXColormap) cm ); 
-    
+    m_backgroundColour.CalcPixel( (WXColormap) cm );
+

     m_foregroundColour = *wxBLACK;
     m_foregroundColour = *wxBLACK;

-    m_foregroundColour.CalcPixel( (WXColormap) cm ); 
+    m_foregroundColour.CalcPixel( (WXColormap) cm );

 
     Window xparent = (Window) parent->GetClientAreaWindow();
 
     Window xparent = (Window) parent->GetClientAreaWindow();

-    
+

     // Add window's own scrollbars to main window, not to client window
     if (parent->GetInsertIntoMain())
     {
         // wxLogDebug( "Inserted into main: %s", GetName().c_str() );
         xparent = (Window) parent->GetMainWindow();
     }
     // Add window's own scrollbars to main window, not to client window
     if (parent->GetInsertIntoMain())
     {
         // wxLogDebug( "Inserted into main: %s", GetName().c_str() );
         xparent = (Window) parent->GetMainWindow();
     }

-    
+

     // Size (not including the border) must be nonzero (or a Value error results)!
     // Note: The Xlib manual doesn't mention this restriction of XCreateWindow.
     wxSize size2(size);
     // Size (not including the border) must be nonzero (or a Value error results)!
     // Note: The Xlib manual doesn't mention this restriction of XCreateWindow.
     wxSize size2(size);


@@ -162,7 +156,7 @@ bool wxWindowX11::Create(wxWindow *parent, wxWindowID id,
         pos2.x = 0;
     if (pos2.y == -1)
         pos2.y = 0;
         pos2.x = 0;
     if (pos2.y == -1)
         pos2.y = 0;

-        
+

 #if wxUSE_TWO_WINDOWS
     bool need_two_windows =
         ((( wxSUNKEN_BORDER | wxRAISED_BORDER | wxSIMPLE_BORDER | wxHSCROLL | wxVSCROLL ) & m_windowStyle) != 0);
 #if wxUSE_TWO_WINDOWS
     bool need_two_windows =
         ((( wxSUNKEN_BORDER | wxRAISED_BORDER | wxSIMPLE_BORDER | wxHSCROLL | wxVSCROLL ) & m_windowStyle) != 0);


@@ -175,76 +169,76 @@ bool wxWindowX11::Create(wxWindow *parent, wxWindowID id,
 #else
     XSetWindowAttributes xattributes;
     long xattributes_mask = 0;
 #else
     XSetWindowAttributes xattributes;
     long xattributes_mask = 0;

-    
+

     xattributes_mask |= CWBackPixel;
     xattributes.background_pixel = m_backgroundColour.GetPixel();
     xattributes_mask |= CWBackPixel;
     xattributes.background_pixel = m_backgroundColour.GetPixel();

-    
+

     xattributes_mask |= CWBorderPixel;
     xattributes.border_pixel = BlackPixel( xdisplay, xscreen );
     xattributes_mask |= CWBorderPixel;
     xattributes.border_pixel = BlackPixel( xdisplay, xscreen );

-    
+

     xattributes_mask |= CWEventMask;
 #endif
     xattributes_mask |= CWEventMask;
 #endif

-    
+

     if (need_two_windows)
     {
 #if wxUSE_NANOX
         long backColor, foreColor;
         backColor = GR_RGB(m_backgroundColour.Red(), m_backgroundColour.Green(), m_backgroundColour.Blue());
         foreColor = GR_RGB(m_foregroundColour.Red(), m_foregroundColour.Green(), m_foregroundColour.Blue());
     if (need_two_windows)
     {
 #if wxUSE_NANOX
         long backColor, foreColor;
         backColor = GR_RGB(m_backgroundColour.Red(), m_backgroundColour.Green(), m_backgroundColour.Blue());
         foreColor = GR_RGB(m_foregroundColour.Red(), m_foregroundColour.Green(), m_foregroundColour.Blue());

-    
-        Window xwindow = XCreateWindowWithColor( xdisplay, xparent, pos2.x, pos2.y, size2.x, size2.y, 
+
+        Window xwindow = XCreateWindowWithColor( xdisplay, xparent, pos2.x, pos2.y, size2.x, size2.y,

                                     0, 0, InputOutput, xvisual, backColor, foreColor);
         XSelectInput( xdisplay, xwindow,
           GR_EVENT_MASK_CLOSE_REQ | ExposureMask | KeyPressMask | KeyReleaseMask | ButtonPressMask | ButtonReleaseMask |
           ButtonMotionMask | EnterWindowMask | LeaveWindowMask | PointerMotionMask |
           KeymapStateMask | FocusChangeMask | ColormapChangeMask | StructureNotifyMask |
                       PropertyChangeMask );
                                     0, 0, InputOutput, xvisual, backColor, foreColor);
         XSelectInput( xdisplay, xwindow,
           GR_EVENT_MASK_CLOSE_REQ | ExposureMask | KeyPressMask | KeyReleaseMask | ButtonPressMask | ButtonReleaseMask |
           ButtonMotionMask | EnterWindowMask | LeaveWindowMask | PointerMotionMask |
           KeymapStateMask | FocusChangeMask | ColormapChangeMask | StructureNotifyMask |
                       PropertyChangeMask );

-        
+

 #else
         // Normal X11
 #else
         // Normal X11

-        xattributes.event_mask = 
+        xattributes.event_mask =

             ExposureMask | StructureNotifyMask | ColormapChangeMask;
 
             ExposureMask | StructureNotifyMask | ColormapChangeMask;
 

-        Window xwindow = XCreateWindow( xdisplay, xparent, pos2.x, pos2.y, size2.x, size2.y, 
+        Window xwindow = XCreateWindow( xdisplay, xparent, pos2.x, pos2.y, size2.x, size2.y,

             0, DefaultDepth(xdisplay,xscreen), InputOutput, xvisual, xattributes_mask, &xattributes );
 
 #endif
             0, DefaultDepth(xdisplay,xscreen), InputOutput, xvisual, xattributes_mask, &xattributes );
 
 #endif

-        
+

         XSetWindowBackgroundPixmap( xdisplay, xwindow, None );
         XSetWindowBackgroundPixmap( xdisplay, xwindow, None );

-    
+

         m_mainWindow = (WXWindow) xwindow;
         wxAddWindowToTable( xwindow, (wxWindow*) this );
         m_mainWindow = (WXWindow) xwindow;
         wxAddWindowToTable( xwindow, (wxWindow*) this );

-    
+

         XMapWindow( xdisplay, xwindow );
 
         XMapWindow( xdisplay, xwindow );
 

-#if !wxUSE_NANOX    
-        xattributes.event_mask = 
+#if !wxUSE_NANOX
+        xattributes.event_mask =

             ExposureMask | KeyPressMask | KeyReleaseMask | ButtonPressMask | ButtonReleaseMask |
             ButtonMotionMask | EnterWindowMask | LeaveWindowMask | PointerMotionMask |
             KeymapStateMask | FocusChangeMask | ColormapChangeMask | StructureNotifyMask |
             PropertyChangeMask | VisibilityChangeMask ;
 
             ExposureMask | KeyPressMask | KeyReleaseMask | ButtonPressMask | ButtonReleaseMask |
             ButtonMotionMask | EnterWindowMask | LeaveWindowMask | PointerMotionMask |
             KeymapStateMask | FocusChangeMask | ColormapChangeMask | StructureNotifyMask |
             PropertyChangeMask | VisibilityChangeMask ;
 

-        if (HasFlag( wxNO_FULL_REPAINT_ON_RESIZE ))
+        if (!HasFlag( wxFULL_REPAINT_ON_RESIZE ))

         {
             xattributes_mask |= CWBitGravity;
             xattributes.bit_gravity = StaticGravity;
         }
 #endif
         {
             xattributes_mask |= CWBitGravity;
             xattributes.bit_gravity = StaticGravity;
         }
 #endif

-        
+

         if (HasFlag( wxSUNKEN_BORDER) || HasFlag( wxRAISED_BORDER))
         {
             pos2.x = 2;
             pos2.y = 2;
             size2.x -= 4;
             size2.y -= 4;
         if (HasFlag( wxSUNKEN_BORDER) || HasFlag( wxRAISED_BORDER))
         {
             pos2.x = 2;
             pos2.y = 2;
             size2.x -= 4;
             size2.y -= 4;

-        } 
+        }

         else if (HasFlag( wxSIMPLE_BORDER ))
         {
             pos2.x = 1;
             pos2.y = 1;
             size2.x -= 2;
             size2.y -= 2;
         else if (HasFlag( wxSIMPLE_BORDER ))
         {
             pos2.x = 1;
             pos2.y = 1;
             size2.x -= 2;
             size2.y -= 2;

-        } 
+        }

         else
         {
             pos2.x = 0;
         else
         {
             pos2.x = 0;


@@ -257,28 +251,28 @@ bool wxWindowX11::Create(wxWindow *parent, wxWindowID id,
         if (size2.y <= 0)
             size2.y = 1;
 
         if (size2.y <= 0)
             size2.y = 1;
 

-#if wxUSE_NANOX        
+#if wxUSE_NANOX

         backColor = GR_RGB(m_backgroundColour.Red(), m_backgroundColour.Green(), m_backgroundColour.Blue());
         foreColor = GR_RGB(m_foregroundColour.Red(), m_foregroundColour.Green(), m_foregroundColour.Blue());
         backColor = GR_RGB(m_backgroundColour.Red(), m_backgroundColour.Green(), m_backgroundColour.Blue());
         foreColor = GR_RGB(m_foregroundColour.Red(), m_foregroundColour.Green(), m_foregroundColour.Blue());

-    
-        xwindow = XCreateWindowWithColor( xdisplay, xwindow, pos2.x, pos2.y, size2.x, size2.y, 
+
+        xwindow = XCreateWindowWithColor( xdisplay, xwindow, pos2.x, pos2.y, size2.x, size2.y,

                                     0, 0, InputOutput, xvisual, backColor, foreColor);
         XSelectInput( xdisplay, xwindow,
           GR_EVENT_MASK_CLOSE_REQ | ExposureMask | KeyPressMask | KeyReleaseMask | ButtonPressMask | ButtonReleaseMask |
           ButtonMotionMask | EnterWindowMask | LeaveWindowMask | PointerMotionMask |
           KeymapStateMask | FocusChangeMask | ColormapChangeMask | StructureNotifyMask |
                       PropertyChangeMask );
                                     0, 0, InputOutput, xvisual, backColor, foreColor);
         XSelectInput( xdisplay, xwindow,
           GR_EVENT_MASK_CLOSE_REQ | ExposureMask | KeyPressMask | KeyReleaseMask | ButtonPressMask | ButtonReleaseMask |
           ButtonMotionMask | EnterWindowMask | LeaveWindowMask | PointerMotionMask |
           KeymapStateMask | FocusChangeMask | ColormapChangeMask | StructureNotifyMask |
                       PropertyChangeMask );

-        
+

 #else
 #else

-        xwindow = XCreateWindow( xdisplay, xwindow, pos2.x, pos2.y, size2.x, size2.y, 
+        xwindow = XCreateWindow( xdisplay, xwindow, pos2.x, pos2.y, size2.x, size2.y,

             0, DefaultDepth(xdisplay,xscreen), InputOutput, xvisual, xattributes_mask, &xattributes );
 #endif
             0, DefaultDepth(xdisplay,xscreen), InputOutput, xvisual, xattributes_mask, &xattributes );
 #endif

-        
+

         XSetWindowBackgroundPixmap( xdisplay, xwindow, None );
         XSetWindowBackgroundPixmap( xdisplay, xwindow, None );

-    
+

         m_clientWindow = (WXWindow) xwindow;
         wxAddClientWindowToTable( xwindow, (wxWindow*) this );
         m_clientWindow = (WXWindow) xwindow;
         wxAddClientWindowToTable( xwindow, (wxWindow*) this );

-        
+

         XMapWindow( xdisplay, xwindow );
     }
     else
         XMapWindow( xdisplay, xwindow );
     }
     else


@@ -288,38 +282,38 @@ bool wxWindowX11::Create(wxWindow *parent, wxWindowID id,
         long backColor, foreColor;
         backColor = GR_RGB(m_backgroundColour.Red(), m_backgroundColour.Green(), m_backgroundColour.Blue());
         foreColor = GR_RGB(m_foregroundColour.Red(), m_foregroundColour.Green(), m_foregroundColour.Blue());
         long backColor, foreColor;
         backColor = GR_RGB(m_backgroundColour.Red(), m_backgroundColour.Green(), m_backgroundColour.Blue());
         foreColor = GR_RGB(m_foregroundColour.Red(), m_foregroundColour.Green(), m_foregroundColour.Blue());

-    
-        Window xwindow = XCreateWindowWithColor( xdisplay, xparent, pos2.x, pos2.y, size2.x, size2.y, 
+
+        Window xwindow = XCreateWindowWithColor( xdisplay, xparent, pos2.x, pos2.y, size2.x, size2.y,

                                     0, 0, InputOutput, xvisual, backColor, foreColor);
         XSelectInput( xdisplay, xwindow,
           GR_EVENT_MASK_CLOSE_REQ | ExposureMask | KeyPressMask | KeyReleaseMask | ButtonPressMask | ButtonReleaseMask |
           ButtonMotionMask | EnterWindowMask | LeaveWindowMask | PointerMotionMask |
           KeymapStateMask | FocusChangeMask | ColormapChangeMask | StructureNotifyMask |
                       PropertyChangeMask );
                                     0, 0, InputOutput, xvisual, backColor, foreColor);
         XSelectInput( xdisplay, xwindow,
           GR_EVENT_MASK_CLOSE_REQ | ExposureMask | KeyPressMask | KeyReleaseMask | ButtonPressMask | ButtonReleaseMask |
           ButtonMotionMask | EnterWindowMask | LeaveWindowMask | PointerMotionMask |
           KeymapStateMask | FocusChangeMask | ColormapChangeMask | StructureNotifyMask |
                       PropertyChangeMask );

-        
+

 #else
 #else

-        xattributes.event_mask = 
+        xattributes.event_mask =

             ExposureMask | KeyPressMask | KeyReleaseMask | ButtonPressMask | ButtonReleaseMask |
             ButtonMotionMask | EnterWindowMask | LeaveWindowMask | PointerMotionMask |
             KeymapStateMask | FocusChangeMask | ColormapChangeMask | StructureNotifyMask |
             PropertyChangeMask | VisibilityChangeMask ;
 
             ExposureMask | KeyPressMask | KeyReleaseMask | ButtonPressMask | ButtonReleaseMask |
             ButtonMotionMask | EnterWindowMask | LeaveWindowMask | PointerMotionMask |
             KeymapStateMask | FocusChangeMask | ColormapChangeMask | StructureNotifyMask |
             PropertyChangeMask | VisibilityChangeMask ;
 

-        if (HasFlag( wxNO_FULL_REPAINT_ON_RESIZE ))
+        if (!HasFlag( wxFULL_REPAINT_ON_RESIZE ))

         {
             xattributes_mask |= CWBitGravity;
             xattributes.bit_gravity = NorthWestGravity;
         }
         {
             xattributes_mask |= CWBitGravity;
             xattributes.bit_gravity = NorthWestGravity;
         }

-        
-        Window xwindow = XCreateWindow( xdisplay, xparent, pos2.x, pos2.y, size2.x, size2.y, 
+
+        Window xwindow = XCreateWindow( xdisplay, xparent, pos2.x, pos2.y, size2.x, size2.y,

             0, DefaultDepth(xdisplay,xscreen), InputOutput, xvisual, xattributes_mask, &xattributes );
 #endif
             0, DefaultDepth(xdisplay,xscreen), InputOutput, xvisual, xattributes_mask, &xattributes );
 #endif

-        
+

         XSetWindowBackgroundPixmap( xdisplay, xwindow, None );
         XSetWindowBackgroundPixmap( xdisplay, xwindow, None );

-    
+

         m_mainWindow = (WXWindow) xwindow;
         m_clientWindow = m_mainWindow;
         wxAddWindowToTable( xwindow, (wxWindow*) this );
         m_mainWindow = (WXWindow) xwindow;
         m_clientWindow = m_mainWindow;
         wxAddWindowToTable( xwindow, (wxWindow*) this );

-        
+

         XMapWindow( xdisplay, xwindow );
     }
 
         XMapWindow( xdisplay, xwindow );
     }
 


@@ -341,13 +335,12 @@ bool wxWindowX11::Create(wxWindow *parent, wxWindowID id,
 // Destructor
 wxWindowX11::~wxWindowX11()
 {
 // Destructor
 wxWindowX11::~wxWindowX11()
 {

+    SendDestroyEvent();
+

     if (g_captureWindow == this)
         g_captureWindow = NULL;
     if (g_captureWindow == this)
         g_captureWindow = NULL;

-    
+

     m_isBeingDeleted = TRUE;
     m_isBeingDeleted = TRUE;

-    
-    if (m_parent)
-        m_parent->RemoveChild( this );

 
     DestroyChildren();
 
 
     DestroyChildren();
 


@@ -359,7 +352,7 @@ wxWindowX11::~wxWindowX11()
         XDestroyWindow( wxGlobalDisplay(), xwindow );
         m_clientWindow = NULL;
     }
         XDestroyWindow( wxGlobalDisplay(), xwindow );
         m_clientWindow = NULL;
     }

-        
+

     // Destroy the window
     Window xwindow = (Window) m_mainWindow;
     wxDeleteWindowFromTable( xwindow );
     // Destroy the window
     Window xwindow = (Window) m_mainWindow;
     wxDeleteWindowFromTable( xwindow );


@@ -374,11 +367,11 @@ wxWindowX11::~wxWindowX11()
 void wxWindowX11::SetFocus()
 {
     Window xwindow = (Window) m_clientWindow;
 void wxWindowX11::SetFocus()
 {
     Window xwindow = (Window) m_clientWindow;

-    
+

     wxCHECK_RET( xwindow, wxT("invalid window") );
     wxCHECK_RET( xwindow, wxT("invalid window") );

-    
+

     wxCHECK_RET( AcceptsFocus(), wxT("set focus on window that doesn't accept the focus") );
     wxCHECK_RET( AcceptsFocus(), wxT("set focus on window that doesn't accept the focus") );

-    
+

 #if 0
     if (GetName() == "scrollBar")
     {
 #if 0
     if (GetName() == "scrollBar")
     {


@@ -386,7 +379,7 @@ void wxWindowX11::SetFocus()
         *crash = 0;
     }
 #endif
         *crash = 0;
     }
 #endif

-    
+

     if (wxWindowIsVisible(xwindow))
     {
         wxLogTrace( _T("focus"), _T("wxWindowX11::SetFocus: %s"), GetClassInfo()->GetClassName());
     if (wxWindowIsVisible(xwindow))
     {
         wxLogTrace( _T("focus"), _T("wxWindowX11::SetFocus: %s"), GetClassInfo()->GetClassName());


@@ -401,7 +394,7 @@ void wxWindowX11::SetFocus()
 }
 
 // Get the window with the focus
 }
 
 // Get the window with the focus

-wxWindow *wxWindowBase::FindFocus()
+wxWindow *wxWindowBase::DoFindFocus()

 {
     Window xfocus = (Window) 0;
     int revert = 0;
 {
     Window xfocus = (Window) 0;
     int revert = 0;


@@ -427,7 +420,7 @@ bool wxWindowX11::Enable(bool enable)
 {
     if ( !wxWindowBase::Enable(enable) )
         return FALSE;
 {
     if ( !wxWindowBase::Enable(enable) )
         return FALSE;

-   
+

     return TRUE;
 }
 
     return TRUE;
 }
 


@@ -469,21 +462,21 @@ void wxWindowX11::DoCaptureMouse()
 {
     if ((g_captureWindow != NULL) && (g_captureWindow != this))
     {
 {
     if ((g_captureWindow != NULL) && (g_captureWindow != this))
     {

-        wxASSERT_MSG(FALSE, "Trying to capture before mouse released.");
+        wxASSERT_MSG(FALSE, wxT("Trying to capture before mouse released."));

 
         // Core dump now
         int *tmp = NULL;
         (*tmp) = 1;
         return;
     }
 
         // Core dump now
         int *tmp = NULL;
         (*tmp) = 1;
         return;
     }

-    
+

     if (m_winCaptured)
         return;
 
     Window xwindow = (Window) m_clientWindow;
 
     wxCHECK_RET( xwindow, wxT("invalid window") );
     if (m_winCaptured)
         return;
 
     Window xwindow = (Window) m_clientWindow;
 
     wxCHECK_RET( xwindow, wxT("invalid window") );

-    
+

     g_captureWindow = (wxWindow*) this;
 
     if (xwindow)
     g_captureWindow = (wxWindow*) this;
 
     if (xwindow)


@@ -504,7 +497,7 @@ void wxWindowX11::DoCaptureMouse()
             wxLogDebug(msg);
             if (res == GrabNotViewable)
                 wxLogDebug( wxT("This is not a viewable window - perhaps not shown yet?") );
             wxLogDebug(msg);
             if (res == GrabNotViewable)
                 wxLogDebug( wxT("This is not a viewable window - perhaps not shown yet?") );

-                
+

             g_captureWindow = NULL;
             return;
         }
             g_captureWindow = NULL;
             return;
         }


@@ -516,7 +509,7 @@ void wxWindowX11::DoCaptureMouse()
 void wxWindowX11::DoReleaseMouse()
 {
     g_captureWindow = NULL;
 void wxWindowX11::DoReleaseMouse()
 {
     g_captureWindow = NULL;

-    
+

     if ( !m_winCaptured )
         return;
 
     if ( !m_winCaptured )
         return;
 


@@ -526,7 +519,7 @@ void wxWindowX11::DoReleaseMouse()
     {
         XUngrabPointer( wxGlobalDisplay(), CurrentTime );
     }
     {
         XUngrabPointer( wxGlobalDisplay(), CurrentTime );
     }

-    
+

     // wxLogDebug( "Ungrabbed pointer in %s", GetName().c_str() );
 
     m_winCaptured = FALSE;
     // wxLogDebug( "Ungrabbed pointer in %s", GetName().c_str() );
 
     m_winCaptured = FALSE;


@@ -554,7 +547,7 @@ bool wxWindowX11::SetCursor(const wxCursor& cursor)
     Window xwindow = (Window) m_clientWindow;
 
     wxCHECK_MSG( xwindow, FALSE, wxT("invalid window") );
     Window xwindow = (Window) m_clientWindow;
 
     wxCHECK_MSG( xwindow, FALSE, wxT("invalid window") );

-    
+

     wxCursor cursorToUse;
     if (m_cursor.Ok())
         cursorToUse = m_cursor;
     wxCursor cursorToUse;
     if (m_cursor.Ok())
         cursorToUse = m_cursor;


@@ -574,7 +567,7 @@ void wxWindowX11::WarpPointer (int x, int y)
     Window xwindow = (Window) m_clientWindow;
 
     wxCHECK_RET( xwindow, wxT("invalid window") );
     Window xwindow = (Window) m_clientWindow;
 
     wxCHECK_RET( xwindow, wxT("invalid window") );

-    
+

     XWarpPointer( wxGlobalDisplay(), None, xwindow, 0, 0, 0, 0, x, y);
 }
 
     XWarpPointer( wxGlobalDisplay(), None, xwindow, 0, 0, 0, 0, x, y);
 }
 


@@ -583,27 +576,27 @@ void wxWindowX11::ScrollWindow(int dx, int dy, const wxRect *rect)
 {
     // No scrolling requested.
     if ((dx == 0) && (dy == 0)) return;
 {
     // No scrolling requested.
     if ((dx == 0) && (dy == 0)) return;

-    
+

     if (!m_updateRegion.IsEmpty())
     {
         m_updateRegion.Offset( dx, dy );
     if (!m_updateRegion.IsEmpty())
     {
         m_updateRegion.Offset( dx, dy );

-        
+

         int cw = 0;
         int ch = 0;
         GetSize( &cw, &ch );  // GetClientSize() ??
         m_updateRegion.Intersect( 0, 0, cw, ch );
     }
         int cw = 0;
         int ch = 0;
         GetSize( &cw, &ch );  // GetClientSize() ??
         m_updateRegion.Intersect( 0, 0, cw, ch );
     }

-    
+

     if (!m_clearRegion.IsEmpty())
     {
         m_clearRegion.Offset( dx, dy );
     if (!m_clearRegion.IsEmpty())
     {
         m_clearRegion.Offset( dx, dy );

-        
+

         int cw = 0;
         int ch = 0;
         GetSize( &cw, &ch );  // GetClientSize() ??
         m_clearRegion.Intersect( 0, 0, cw, ch );
     }
         int cw = 0;
         int ch = 0;
         GetSize( &cw, &ch );  // GetClientSize() ??
         m_clearRegion.Intersect( 0, 0, cw, ch );
     }

-    
+

     Window xwindow = (Window) GetClientAreaWindow();
 
     wxCHECK_RET( xwindow, wxT("invalid window") );
     Window xwindow = (Window) GetClientAreaWindow();
 
     wxCHECK_RET( xwindow, wxT("invalid window") );


@@ -621,7 +614,7 @@ void wxWindowX11::ScrollWindow(int dx, int dy, const wxRect *rect)
     {
         s_x = rect->x;
         s_y = rect->y;
     {
         s_x = rect->x;
         s_y = rect->y;

-        
+

         cw = rect->width;
         ch = rect->height;
     }
         cw = rect->width;
         ch = rect->height;
     }


@@ -631,7 +624,7 @@ void wxWindowX11::ScrollWindow(int dx, int dy, const wxRect *rect)
         s_y = 0;
         GetClientSize( &cw, &ch );
     }
         s_y = 0;
         GetClientSize( &cw, &ch );
     }

-    
+

 #if wxUSE_TWO_WINDOWS
     wxPoint offset( 0,0 );
 #else
 #if wxUSE_TWO_WINDOWS
     wxPoint offset( 0,0 );
 #else


@@ -639,10 +632,10 @@ void wxWindowX11::ScrollWindow(int dx, int dy, const wxRect *rect)
     s_x += offset.x;
     s_y += offset.y;
 #endif
     s_x += offset.x;
     s_y += offset.y;
 #endif

-        
+

     int w = cw - abs(dx);
     int h = ch - abs(dy);
     int w = cw - abs(dx);
     int h = ch - abs(dy);

-        
+

     if ((h < 0) || (w < 0))
     {
         Refresh();
     if ((h < 0) || (w < 0))
     {
         Refresh();


@@ -654,17 +647,17 @@ void wxWindowX11::ScrollWindow(int dx, int dy, const wxRect *rect)
         if (dy < 0) rect.y = ch+dy + offset.y; else rect.y = s_y;
         if (dy != 0) rect.width = cw; else rect.width = abs(dx);
         if (dx != 0) rect.height = ch; else rect.height = abs(dy);
         if (dy < 0) rect.y = ch+dy + offset.y; else rect.y = s_y;
         if (dy != 0) rect.width = cw; else rect.width = abs(dx);
         if (dx != 0) rect.height = ch; else rect.height = abs(dy);

-    
+

         int d_x = s_x;
         int d_y = s_y;
         int d_x = s_x;
         int d_y = s_y;

-        
+

         if (dx < 0) s_x += -dx;
         if (dy < 0) s_y += -dy;
         if (dx > 0) d_x = dx + offset.x;
         if (dy > 0) d_y = dy + offset.y;
 
         XCopyArea( xdisplay, xwindow, xwindow, xgc, s_x, s_y, w, h, d_x, d_y );
         if (dx < 0) s_x += -dx;
         if (dy < 0) s_y += -dy;
         if (dx > 0) d_x = dx + offset.x;
         if (dy > 0) d_y = dy + offset.y;
 
         XCopyArea( xdisplay, xwindow, xwindow, xgc, s_x, s_y, w, h, d_x, d_y );

-        
+

         // wxLogDebug( "Copy: s_x %d s_y %d w %d h %d d_x %d d_y %d", s_x, s_y, w, h, d_x, d_y );
 
         // wxLogDebug( "Update: %d %d %d %d", rect.x, rect.y, rect.width, rect.height );
         // wxLogDebug( "Copy: s_x %d s_y %d w %d h %d d_x %d d_y %d", s_x, s_y, w, h, d_x, d_y );
 
         // wxLogDebug( "Update: %d %d %d %d", rect.x, rect.y, rect.width, rect.height );


@@ -672,7 +665,7 @@ void wxWindowX11::ScrollWindow(int dx, int dy, const wxRect *rect)
         m_updateRegion.Union( rect );
         m_clearRegion.Union( rect );
     }
         m_updateRegion.Union( rect );
         m_clearRegion.Union( rect );
     }

-    
+

     XFreeGC( xdisplay, xgc );
 }
 
     XFreeGC( xdisplay, xgc );
 }
 


@@ -723,13 +716,13 @@ void wxWindowX11::DoGetSize(int *x, int *y) const
     Window xwindow = (Window) m_mainWindow;
 
     wxCHECK_RET( xwindow, wxT("invalid window") );
     Window xwindow = (Window) m_mainWindow;
 
     wxCHECK_RET( xwindow, wxT("invalid window") );

-    
+

     //XSync(wxGlobalDisplay(), False);
 
     XWindowAttributes attr;
     Status status = XGetWindowAttributes( wxGlobalDisplay(), xwindow, &attr );
     wxASSERT(status);
     //XSync(wxGlobalDisplay(), False);
 
     XWindowAttributes attr;
     Status status = XGetWindowAttributes( wxGlobalDisplay(), xwindow, &attr );
     wxASSERT(status);

-        
+

     if (status)
     {
         *x = attr.width /* + 2*m_borderSize */ ;
     if (status)
     {
         *x = attr.width /* + 2*m_borderSize */ ;


@@ -746,12 +739,12 @@ void wxWindowX11::DoGetPosition(int *x, int *y) const
         XWindowAttributes attr;
         Status status = XGetWindowAttributes(wxGlobalDisplay(), window, & attr);
         wxASSERT(status);
         XWindowAttributes attr;
         Status status = XGetWindowAttributes(wxGlobalDisplay(), window, & attr);
         wxASSERT(status);

-        
+

         if (status)
         {
             *x = attr.x;
             *y = attr.y;
         if (status)
         {
             *x = attr.x;
             *y = attr.y;

-            
+

             // We may be faking the client origin. So a window that's really at (0, 30)
             // may appear (to wxWin apps) to be at (0, 0).
             if (GetParent())
             // We may be faking the client origin. So a window that's really at (0, 30)
             // may appear (to wxWin apps) to be at (0, 0).
             if (GetParent())


@@ -799,7 +792,7 @@ void wxWindowX11::DoGetClientSize(int *x, int *y) const
         XWindowAttributes attr;
         Status status = XGetWindowAttributes( wxGlobalDisplay(), window, &attr );
         wxASSERT(status);
         XWindowAttributes attr;
         Status status = XGetWindowAttributes( wxGlobalDisplay(), window, &attr );
         wxASSERT(status);

-        
+

         if (status)
         {
             *x = attr.width ;
         if (status)
         {
             *x = attr.width ;


@@ -811,7 +804,7 @@ void wxWindowX11::DoGetClientSize(int *x, int *y) const
 void wxWindowX11::DoSetSize(int x, int y, int width, int height, int sizeFlags)
 {
     //    wxLogDebug("DoSetSize: %s (%ld) %d, %d %dx%d", GetClassInfo()->GetClassName(), GetId(), x, y, width, height);
 void wxWindowX11::DoSetSize(int x, int y, int width, int height, int sizeFlags)
 {
     //    wxLogDebug("DoSetSize: %s (%ld) %d, %d %dx%d", GetClassInfo()->GetClassName(), GetId(), x, y, width, height);

-    
+

     Window xwindow = (Window) m_mainWindow;
 
     wxCHECK_RET( xwindow, wxT("invalid window") );
     Window xwindow = (Window) m_mainWindow;
 
     wxCHECK_RET( xwindow, wxT("invalid window") );


@@ -819,12 +812,12 @@ void wxWindowX11::DoSetSize(int x, int y, int width, int height, int sizeFlags)
     XWindowAttributes attr;
     Status status = XGetWindowAttributes( wxGlobalDisplay(), xwindow, &attr );
     wxCHECK_RET( status, wxT("invalid window attributes") );
     XWindowAttributes attr;
     Status status = XGetWindowAttributes( wxGlobalDisplay(), xwindow, &attr );
     wxCHECK_RET( status, wxT("invalid window attributes") );

-        
+

     int new_x = attr.x;
     int new_y = attr.y;
     int new_w = attr.width;
     int new_h = attr.height;
     int new_x = attr.x;
     int new_y = attr.y;
     int new_w = attr.width;
     int new_h = attr.height;

-    
+

     if (x != -1 || (sizeFlags & wxSIZE_ALLOW_MINUS_ONE))
     {
         int yy = 0;
     if (x != -1 || (sizeFlags & wxSIZE_ALLOW_MINUS_ONE))
     {
         int yy = 0;


@@ -849,24 +842,24 @@ void wxWindowX11::DoSetSize(int x, int y, int width, int height, int sizeFlags)
         if (new_h <= 0)
             new_h = 20;
     }
         if (new_h <= 0)
             new_h = 20;
     }

-    
+

     DoMoveWindow( new_x, new_y, new_w, new_h );
 }
 
 void wxWindowX11::DoSetClientSize(int width, int height)
 {
     //    wxLogDebug("DoSetClientSize: %s (%ld) %dx%d", GetClassInfo()->GetClassName(), GetId(), width, height);
     DoMoveWindow( new_x, new_y, new_w, new_h );
 }
 
 void wxWindowX11::DoSetClientSize(int width, int height)
 {
     //    wxLogDebug("DoSetClientSize: %s (%ld) %dx%d", GetClassInfo()->GetClassName(), GetId(), width, height);

-    
+

     Window xwindow = (Window) m_mainWindow;
 
     wxCHECK_RET( xwindow, wxT("invalid window") );
 
     XResizeWindow( wxGlobalDisplay(), xwindow, width, height );
     Window xwindow = (Window) m_mainWindow;
 
     wxCHECK_RET( xwindow, wxT("invalid window") );
 
     XResizeWindow( wxGlobalDisplay(), xwindow, width, height );

-    
+

     if (m_mainWindow != m_clientWindow)
     {
         xwindow = (Window) m_clientWindow;
     if (m_mainWindow != m_clientWindow)
     {
         xwindow = (Window) m_clientWindow;

-        
+

         wxWindow *window = (wxWindow*) this;
         wxRenderer *renderer = window->GetRenderer();
         if (renderer)
         wxWindow *window = (wxWindow*) this;
         wxRenderer *renderer = window->GetRenderer();
         if (renderer)


@@ -875,7 +868,7 @@ void wxWindowX11::DoSetClientSize(int width, int height)
             width -= border.x + border.width;
             height -= border.y + border.height;
         }
             width -= border.x + border.width;
             height -= border.y + border.height;
         }

-        
+

         XResizeWindow( wxGlobalDisplay(), xwindow, width, height );
     }
 }
         XResizeWindow( wxGlobalDisplay(), xwindow, width, height );
     }
 }


@@ -899,7 +892,7 @@ void wxWindowX11::DoMoveWindow(int x, int y, int width, int height)
     if (m_mainWindow != m_clientWindow)
     {
         xwindow = (Window) m_clientWindow;
     if (m_mainWindow != m_clientWindow)
     {
         xwindow = (Window) m_clientWindow;

-        
+

         wxWindow *window = (wxWindow*) this;
         wxRenderer *renderer = window->GetRenderer();
         if (renderer)
         wxWindow *window = (wxWindow*) this;
         wxRenderer *renderer = window->GetRenderer();
         if (renderer)


@@ -915,7 +908,7 @@ void wxWindowX11::DoMoveWindow(int x, int y, int width, int height)
             x = 0;
             y = 0;
         }
             x = 0;
             y = 0;
         }

-        
+

         wxScrollBar *sb = window->GetScrollbar( wxHORIZONTAL );
         if (sb && sb->IsShown())
         {
         wxScrollBar *sb = window->GetScrollbar( wxHORIZONTAL );
         if (sb && sb->IsShown())
         {


@@ -928,11 +921,11 @@ void wxWindowX11::DoMoveWindow(int x, int y, int width, int height)
             wxSize size = sb->GetSize();
             width -= size.x;
         }
             wxSize size = sb->GetSize();
             width -= size.x;
         }

-        
+

         XMoveResizeWindow( wxGlobalDisplay(), xwindow, x, y, wxMax(1, width), wxMax(1, height) );
     }
         XMoveResizeWindow( wxGlobalDisplay(), xwindow, x, y, wxMax(1, width), wxMax(1, height) );
     }

-    
-#else    
+
+#else

 
     XWindowChanges windowChanges;
     windowChanges.x = x;
 
     XWindowChanges windowChanges;
     windowChanges.x = x;


@@ -943,11 +936,11 @@ void wxWindowX11::DoMoveWindow(int x, int y, int width, int height)
     int valueMask = CWX | CWY | CWWidth | CWHeight;
 
     XConfigureWindow( wxGlobalDisplay(), xwindow, valueMask, &windowChanges );
     int valueMask = CWX | CWY | CWWidth | CWHeight;
 
     XConfigureWindow( wxGlobalDisplay(), xwindow, valueMask, &windowChanges );

-    
+

 #endif
 }
 
 #endif
 }
 

-void wxWindowX11::SetSizeHints(int minW, int minH, int maxW, int maxH, int incW, int incH)
+void wxWindowX11::DoSetSizeHints(int minW, int minH, int maxW, int maxH, int incW, int incH)

 {
     m_minWidth = minW;
     m_minHeight = minH;
 {
     m_minWidth = minW;
     m_minHeight = minH;


@@ -957,7 +950,7 @@ void wxWindowX11::SetSizeHints(int minW, int minH, int maxW, int maxH, int incW,
 #if !wxUSE_NANOX
     XSizeHints sizeHints;
     sizeHints.flags = 0;
 #if !wxUSE_NANOX
     XSizeHints sizeHints;
     sizeHints.flags = 0;

-    
+

     if (minW > -1 && minH > -1)
     {
         sizeHints.flags |= PMinSize;
     if (minW > -1 && minH > -1)
     {
         sizeHints.flags |= PMinSize;


@@ -987,7 +980,7 @@ void wxWindowX11::SetSizeHints(int minW, int minH, int maxW, int maxH, int incW,
 
 int wxWindowX11::GetCharHeight() const
 {
 
 int wxWindowX11::GetCharHeight() const
 {

-    wxCHECK_MSG( m_font.Ok(), 0, "valid window font needed" );
+    wxCHECK_MSG( m_font.Ok(), 0, wxT("valid window font needed") );

 
 #if wxUSE_UNICODE
     // There should be an easier way.
 
 #if wxUSE_UNICODE
     // There should be an easier way.


@@ -997,7 +990,7 @@ int wxWindowX11::GetCharHeight() const
     int w,h;
     pango_layout_get_pixel_size(layout, &w, &h);
     g_object_unref( G_OBJECT( layout ) );
     int w,h;
     pango_layout_get_pixel_size(layout, &w, &h);
     g_object_unref( G_OBJECT( layout ) );

-    
+

     return h;
 #else
     WXFontStructPtr pFontStruct = m_font.GetFontStruct(1.0, wxGlobalDisplay());
     return h;
 #else
     WXFontStructPtr pFontStruct = m_font.GetFontStruct(1.0, wxGlobalDisplay());


@@ -1014,7 +1007,7 @@ int wxWindowX11::GetCharHeight() const
 
 int wxWindowX11::GetCharWidth() const
 {
 
 int wxWindowX11::GetCharWidth() const
 {

-    wxCHECK_MSG( m_font.Ok(), 0, "valid window font needed" );
+    wxCHECK_MSG( m_font.Ok(), 0, wxT("valid window font needed") );

 
 #if wxUSE_UNICODE
     // There should be an easier way.
 
 #if wxUSE_UNICODE
     // There should be an easier way.


@@ -1024,7 +1017,7 @@ int wxWindowX11::GetCharWidth() const
     int w,h;
     pango_layout_get_pixel_size(layout, &w, &h);
     g_object_unref( G_OBJECT( layout ) );
     int w,h;
     pango_layout_get_pixel_size(layout, &w, &h);
     g_object_unref( G_OBJECT( layout ) );

-    
+

     return w;
 #else
     WXFontStructPtr pFontStruct = m_font.GetFontStruct(1.0, wxGlobalDisplay());
     return w;
 #else
     WXFontStructPtr pFontStruct = m_font.GetFontStruct(1.0, wxGlobalDisplay());


@@ -1057,19 +1050,19 @@ void wxWindowX11::GetTextExtent(const wxString& string,
 
 #if wxUSE_UNICODE
     PangoLayout *layout = pango_layout_new( wxTheApp->GetPangoContext() );
 
 #if wxUSE_UNICODE
     PangoLayout *layout = pango_layout_new( wxTheApp->GetPangoContext() );

-    
+

     PangoFontDescription *desc = fontToUse.GetNativeFontInfo()->description;
     pango_layout_set_font_description(layout, desc);
     PangoFontDescription *desc = fontToUse.GetNativeFontInfo()->description;
     pango_layout_set_font_description(layout, desc);

-    
+

     const wxCharBuffer data = wxConvUTF8.cWC2MB( string );
     pango_layout_set_text(layout, (const char*) data, strlen( (const char*) data ));
     const wxCharBuffer data = wxConvUTF8.cWC2MB( string );
     pango_layout_set_text(layout, (const char*) data, strlen( (const char*) data ));

-        
+

     PangoLayoutLine *line = (PangoLayoutLine *)pango_layout_get_lines(layout)->data;
 
     PangoLayoutLine *line = (PangoLayoutLine *)pango_layout_get_lines(layout)->data;
 

-    
+

     PangoRectangle rect;
     pango_layout_line_get_extents(line, NULL, &rect);
     PangoRectangle rect;
     pango_layout_line_get_extents(line, NULL, &rect);

-    
+

     if (x) (*x) = (wxCoord) (rect.width / PANGO_SCALE);
     if (y) (*y) = (wxCoord) (rect.height / PANGO_SCALE);
     if (descent)
     if (x) (*x) = (wxCoord) (rect.width / PANGO_SCALE);
     if (y) (*y) = (wxCoord) (rect.height / PANGO_SCALE);
     if (descent)


@@ -1118,7 +1111,7 @@ void wxWindowX11::Refresh(bool eraseBack, const wxRect *rect)
         {
             int height,width;
             GetSize( &width, &height );
         {
             int height,width;
             GetSize( &width, &height );

-            
+

             // Schedule for later Updating in ::Update() or ::OnInternalIdle().
             m_clearRegion.Clear();
             m_clearRegion.Union( 0, 0, width, height );
             // Schedule for later Updating in ::Update() or ::OnInternalIdle().
             m_clearRegion.Clear();
             m_clearRegion.Union( 0, 0, width, height );


@@ -1134,7 +1127,7 @@ void wxWindowX11::Refresh(bool eraseBack, const wxRect *rect)
     {
         int height,width;
         GetSize( &width, &height );
     {
         int height,width;
         GetSize( &width, &height );

-            
+

         // Schedule for later Updating in ::Update() or ::OnInternalIdle().
         m_updateRegion.Clear();
         m_updateRegion.Union( 0, 0, width, height );
         // Schedule for later Updating in ::Update() or ::OnInternalIdle().
         m_updateRegion.Clear();
         m_updateRegion.Union( 0, 0, width, height );


@@ -1149,33 +1142,25 @@ void wxWindowX11::Update()
         // Send nc paint events.
         SendNcPaintEvents();
     }
         // Send nc paint events.
         SendNcPaintEvents();
     }

-        
+

     if (!m_updateRegion.IsEmpty())
     {
         // wxLogDebug("wxWindowX11::Update: %s", GetClassInfo()->GetClassName());
         // Actually send erase events.
         SendEraseEvents();
     if (!m_updateRegion.IsEmpty())
     {
         // wxLogDebug("wxWindowX11::Update: %s", GetClassInfo()->GetClassName());
         // Actually send erase events.
         SendEraseEvents();

-        
+

         // Actually send paint events.
         SendPaintEvents();
     }
 }
 
         // Actually send paint events.
         SendPaintEvents();
     }
 }
 

-void wxWindowX11::Clear()
-{
-//    wxClientDC dc((wxWindow*) this);
-//    wxBrush brush(GetBackgroundColour(), wxSOLID);
-//    dc.SetBackground(brush);
-//    dc.Clear();
-}
-

 void wxWindowX11::SendEraseEvents()
 {
     if (m_clearRegion.IsEmpty()) return;
 void wxWindowX11::SendEraseEvents()
 {
     if (m_clearRegion.IsEmpty()) return;

-    
+

     wxClientDC dc( (wxWindow*)this );
     dc.SetClippingRegion( m_clearRegion );
     wxClientDC dc( (wxWindow*)this );
     dc.SetClippingRegion( m_clearRegion );

-    
+

     wxEraseEvent erase_event( GetId(), &dc );
     erase_event.SetEventObject( this );
 
     wxEraseEvent erase_event( GetId(), &dc );
     erase_event.SetEventObject( this );
 


@@ -1184,7 +1169,7 @@ void wxWindowX11::SendEraseEvents()
         Display *xdisplay = wxGlobalDisplay();
         Window xwindow = (Window) GetClientAreaWindow();
         XSetForeground( xdisplay, g_eraseGC, m_backgroundColour.GetPixel() );
         Display *xdisplay = wxGlobalDisplay();
         Window xwindow = (Window) GetClientAreaWindow();
         XSetForeground( xdisplay, g_eraseGC, m_backgroundColour.GetPixel() );

-        
+

         wxRegionIterator upd( m_clearRegion );
         while (upd)
         {
         wxRegionIterator upd( m_clearRegion );
         while (upd)
         {


@@ -1193,7 +1178,7 @@ void wxWindowX11::SendEraseEvents()
             upd ++;
         }
     }
             upd ++;
         }
     }

-    
+

     m_clearRegion.Clear();
 }
 
     m_clearRegion.Clear();
 }
 


@@ -1202,13 +1187,13 @@ void wxWindowX11::SendPaintEvents()
     //    wxLogDebug("SendPaintEvents: %s (%ld)", GetClassInfo()->GetClassName(), GetId());
 
     m_clipPaintRegion = TRUE;
     //    wxLogDebug("SendPaintEvents: %s (%ld)", GetClassInfo()->GetClassName(), GetId());
 
     m_clipPaintRegion = TRUE;

-    
+

     wxPaintEvent paint_event( GetId() );
     paint_event.SetEventObject( this );
     GetEventHandler()->ProcessEvent( paint_event );
     wxPaintEvent paint_event( GetId() );
     paint_event.SetEventObject( this );
     GetEventHandler()->ProcessEvent( paint_event );

-    
+

     m_updateRegion.Clear();
     m_updateRegion.Clear();

-    
+

     m_clipPaintRegion = FALSE;
 }
 
     m_clipPaintRegion = FALSE;
 }
 


@@ -1226,7 +1211,7 @@ void wxWindowX11::SendNcPaintEvents()
     {
         height = sb->GetSize().y;
         y = sb->GetPosition().y;
     {
         height = sb->GetSize().y;
         y = sb->GetPosition().y;

-        
+

         sb = window->GetScrollbar( wxVERTICAL );
         if (sb && sb->IsShown())
         {
         sb = window->GetScrollbar( wxVERTICAL );
         if (sb && sb->IsShown())
         {


@@ -1238,17 +1223,17 @@ void wxWindowX11::SendNcPaintEvents()
             Colormap cm = (Colormap) wxTheApp->GetMainColormap( wxGetDisplay() );
             wxColour colour = wxSystemSettings::GetColour(wxSYS_COLOUR_APPWORKSPACE);
             colour.CalcPixel( (WXColormap) cm );
             Colormap cm = (Colormap) wxTheApp->GetMainColormap( wxGetDisplay() );
             wxColour colour = wxSystemSettings::GetColour(wxSYS_COLOUR_APPWORKSPACE);
             colour.CalcPixel( (WXColormap) cm );

-            
+

             XSetForeground( xdisplay, g_eraseGC, colour.GetPixel() );
             XSetForeground( xdisplay, g_eraseGC, colour.GetPixel() );

-        
+

             XFillRectangle( xdisplay, xwindow, g_eraseGC, x, y, width, height );
         }
     }
             XFillRectangle( xdisplay, xwindow, g_eraseGC, x, y, width, height );
         }
     }

-        
+

     wxNcPaintEvent nc_paint_event( GetId() );
     nc_paint_event.SetEventObject( this );
     GetEventHandler()->ProcessEvent( nc_paint_event );
     wxNcPaintEvent nc_paint_event( GetId() );
     nc_paint_event.SetEventObject( this );
     GetEventHandler()->ProcessEvent( nc_paint_event );

-    
+

     m_updateNcArea = FALSE;
 }
 
     m_updateNcArea = FALSE;
 }
 


@@ -1259,7 +1244,7 @@ void wxWindowX11::SendNcPaintEvents()
 // Responds to colour changes: passes event on to children.
 void wxWindowX11::OnSysColourChanged(wxSysColourChangedEvent& event)
 {
 // Responds to colour changes: passes event on to children.
 void wxWindowX11::OnSysColourChanged(wxSysColourChangedEvent& event)
 {

-    wxWindowList::Node *node = GetChildren().GetFirst();
+    wxWindowList::compatibility_iterator node = GetChildren().GetFirst();

     while ( node )
     {
         // Only propagate to non-top-level windows
     while ( node )
     {
         // Only propagate to non-top-level windows


@@ -1282,10 +1267,11 @@ void wxWindowX11::OnInternalIdle()
 {
     // Update invalidated regions.
     Update();
 {
     // Update invalidated regions.
     Update();

-    
+

     // This calls the UI-update mechanism (querying windows for
     // menu/toolbar/control state information)
     // This calls the UI-update mechanism (querying windows for
     // menu/toolbar/control state information)

-    UpdateWindowUI();
+    if (wxUpdateUIEvent::CanUpdate((wxWindow*) this))
+        UpdateWindowUI(wxUPDATE_UI_FROMIDLE);

 
     // Set the input focus if couldn't do it before
     if (m_needsInputFocus)
 
     // Set the input focus if couldn't do it before
     if (m_needsInputFocus)


@@ -1296,7 +1282,7 @@ void wxWindowX11::OnInternalIdle()
         printf(msg.c_str());
 #endif
         SetFocus();
         printf(msg.c_str());
 #endif
         SetFocus();

-        
+

         // If it couldn't set the focus now, there's
         // no point in trying again.
         m_needsInputFocus = FALSE;
         // If it couldn't set the focus now, there's
         // no point in trying again.
         m_needsInputFocus = FALSE;


@@ -1305,73 +1291,71 @@ void wxWindowX11::OnInternalIdle()
 }
 
 // ----------------------------------------------------------------------------
 }
 
 // ----------------------------------------------------------------------------

-// function which maintain the global hash table mapping Widgets to wxWindows
+// function which maintain the global hash table mapping Widgets to wxWidgets

 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
 

-bool wxAddWindowToTable(Window w, wxWindow *win)
+static bool DoAddWindowToTable(wxWindowHash *hash, Window w, wxWindow *win)

 {
 {

-    wxWindow *oldItem = NULL;
-    if ((oldItem = (wxWindow *)wxWidgetHashTable->Get ((long) w)))
+    if ( !hash->insert(wxWindowHash::value_type(w, win)).second )

     {
     {

-        wxLogDebug( wxT("Widget table clash: new widget is %ld, %s"),
-                    (long)w, win->GetClassInfo()->GetClassName());
+        wxLogDebug( wxT("Widget table clash: new widget is 0x%08x, %s"),
+                    (unsigned int)w, win->GetClassInfo()->GetClassName());

         return FALSE;
     }
 
         return FALSE;
     }
 

-    wxWidgetHashTable->Put((long) w, win);
-

     wxLogTrace( wxT("widget"), wxT("XWindow 0x%08x <-> window %p (%s)"),
                 (unsigned int) w, win, win->GetClassInfo()->GetClassName());
 
     return TRUE;
 }
 
     wxLogTrace( wxT("widget"), wxT("XWindow 0x%08x <-> window %p (%s)"),
                 (unsigned int) w, win, win->GetClassInfo()->GetClassName());
 
     return TRUE;
 }
 

-wxWindow *wxGetWindowFromTable(Window w)
+static inline wxWindow *DoGetWindowFromTable(wxWindowHash *hash, Window w)

 {
 {

-    return (wxWindow *)wxWidgetHashTable->Get((long) w);
+    wxWindowHash::iterator i = hash->find(w);
+    return i == hash->end() ? NULL : i->second;

 }
 
 }
 

-void wxDeleteWindowFromTable(Window w)
+static inline void DoDeleteWindowFromTable(wxWindowHash *hash, Window w)

 {
 {

-    wxWidgetHashTable->Delete((long)w);
+    wxLogTrace( wxT("widget"), wxT("XWindow 0x%08x deleted"), (unsigned int) w);
+
+    hash->erase(w);

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

-// function which maintain the global hash table mapping client widgets
+// public wrappers

 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
 

-bool wxAddClientWindowToTable(Window w, wxWindow *win)
+bool wxAddWindowToTable(Window w, wxWindow *win)

 {
 {

-    wxWindow *oldItem = NULL;
-    if ((oldItem = (wxWindow *)wxClientWidgetHashTable->Get ((long) w)))
-    {
-        wxLogDebug( wxT("Client window table clash: new window is %ld, %s"),
-                    (long)w, win->GetClassInfo()->GetClassName());
-        return FALSE;
-    }
+    return DoAddWindowToTable(wxWidgetHashTable, w, win);
+}

 
 

-    wxClientWidgetHashTable->Put((long) w, win);
+wxWindow *wxGetWindowFromTable(Window w)
+{
+    return DoGetWindowFromTable(wxWidgetHashTable, w);
+}

 
 

-    wxLogTrace( wxT("widget"), wxT("XWindow 0x%08x <-> window %p (%s)"),
-                (unsigned int) w, win, win->GetClassInfo()->GetClassName());
+void wxDeleteWindowFromTable(Window w)
+{
+    DoDeleteWindowFromTable(wxWidgetHashTable, w);
+}

 
 

-    return TRUE;
+bool wxAddClientWindowToTable(Window w, wxWindow *win)
+{
+    return DoAddWindowToTable(wxClientWidgetHashTable, w, win);

 }
 
 wxWindow *wxGetClientWindowFromTable(Window w)
 {
 }
 
 wxWindow *wxGetClientWindowFromTable(Window w)
 {

-    return (wxWindow *)wxClientWidgetHashTable->Get((long) w);
+    return DoGetWindowFromTable(wxClientWidgetHashTable, w);

 }
 
 void wxDeleteClientWindowFromTable(Window w)
 {
 }
 
 void wxDeleteClientWindowFromTable(Window w)
 {

-    wxClientWidgetHashTable->Delete((long)w);
+    DoDeleteWindowFromTable(wxClientWidgetHashTable, w);

 }
 
 }
 

-// ----------------------------------------------------------------------------
-// add/remove window from the table
-// ----------------------------------------------------------------------------
-

 // ----------------------------------------------------------------------------
 // X11-specific accessors
 // ----------------------------------------------------------------------------
 // ----------------------------------------------------------------------------
 // X11-specific accessors
 // ----------------------------------------------------------------------------


@@ -1568,11 +1552,11 @@ bool wxWindowX11::SetBackgroundColour(const wxColour& col)
     Colormap cm = DefaultColormap( xdisplay, xscreen );
 
     m_backgroundColour.CalcPixel( (WXColormap) cm );
     Colormap cm = DefaultColormap( xdisplay, xscreen );
 
     m_backgroundColour.CalcPixel( (WXColormap) cm );

-    
+

     // We don't set the background colour as we paint
     // the background ourselves.
     // XSetWindowBackground( xdisplay, (Window) m_clientWindow, m_backgroundColour.GetPixel() );
     // We don't set the background colour as we paint
     // the background ourselves.
     // XSetWindowBackground( xdisplay, (Window) m_clientWindow, m_backgroundColour.GetPixel() );

-    
+

     return TRUE;
 }
 
     return TRUE;
 }
 


@@ -1591,7 +1575,7 @@ bool wxWindowX11::SetForegroundColour(const wxColour& col)
 wxWindow *wxGetActiveWindow()
 {
     // TODO
 wxWindow *wxGetActiveWindow()
 {
     // TODO

-    wxFAIL_MSG("Not implemented");
+    wxFAIL_MSG(wxT("Not implemented"));

     return NULL;
 }
 
     return NULL;
 }
 


@@ -1662,7 +1646,7 @@ bool wxWinModule::OnInit()
     Window xroot = RootWindow( xdisplay, xscreen );
     g_eraseGC = XCreateGC( xdisplay, xroot, 0, NULL );
     XSetFillStyle( xdisplay, g_eraseGC, FillSolid );
     Window xroot = RootWindow( xdisplay, xscreen );
     g_eraseGC = XCreateGC( xdisplay, xroot, 0, NULL );
     XSetFillStyle( xdisplay, g_eraseGC, FillSolid );

-    
+

     return TRUE;
 }
 
     return TRUE;
 }