Compilation fix.

[wxWidgets.git] / include / wx / msw / subwin.h

diff --git a/include/wx/msw/subwin.h b/include/wx/msw/subwin.h
index c32cade9b86e4b3a85ce933872542dbe3758941c..06ba1433e59b40f291406820a8f808b09319c1c6 100644 (file)
--- a/include/wx/msw/subwin.h
+++ b/include/wx/msw/subwin.h
@@ -32,6 +32,7 @@ public:
 
         m_count = n;
         m_hwnds = (HWND *)calloc(n, sizeof(HWND));
 
         m_count = n;
         m_hwnds = (HWND *)calloc(n, sizeof(HWND));

+        m_ids = new wxWindowIDRef[n];

     }
 
     // non-virtual dtor, this class is not supposed to be used polymorphically
     }
 
     // non-virtual dtor, this class is not supposed to be used polymorphically


@@ -39,10 +40,12 @@ public:
     {
         for ( size_t n = 0; n < m_count; n++ )
         {
     {
         for ( size_t n = 0; n < m_count; n++ )
         {

-            ::DestroyWindow(m_hwnds[n]);
+            if ( m_hwnds[n] )
+                ::DestroyWindow(m_hwnds[n]);

         }
 
         free(m_hwnds);
         }
 
         free(m_hwnds);

+        delete [] m_ids;

     }
 
     // get the number of subwindows
     }
 
     // get the number of subwindows


@@ -56,12 +59,21 @@ public:
         return m_hwnds[n];
     }
 
         return m_hwnds[n];
     }
 

-    HWND& operator[](size_t n) { return Get(n); }

     HWND operator[](size_t n) const
     {
         return wx_const_cast(wxSubwindows *, this)->Get(n);
     }
 
     HWND operator[](size_t n) const
     {
         return wx_const_cast(wxSubwindows *, this)->Get(n);
     }
 

+    // initialize the given window: id will be stored in wxWindowIDRef ensuring
+    // that it is not reused while this object exists
+    void Set(size_t n, HWND hwnd, wxWindowID id)
+    {
+        wxASSERT_MSG( n < m_count, _T("subwindow index out of range") );
+
+        m_hwnds[n] = hwnd;
+        m_ids[n] = id;
+    }
+

     // check if we have this window
     bool HasWindow(HWND hwnd)
     {
     // check if we have this window
     bool HasWindow(HWND hwnd)
     {


@@ -84,7 +96,8 @@ public:
         int sw = show ? SW_SHOW : SW_HIDE;
         for ( size_t n = 0; n < m_count; n++ )
         {
         int sw = show ? SW_SHOW : SW_HIDE;
         for ( size_t n = 0; n < m_count; n++ )
         {

-            ::ShowWindow(m_hwnds[n], sw);
+            if ( m_hwnds[n] )
+                ::ShowWindow(m_hwnds[n], sw);

         }
     }
 
         }
     }
 


@@ -93,7 +106,8 @@ public:
     {
         for ( size_t n = 0; n < m_count; n++ )
         {
     {
         for ( size_t n = 0; n < m_count; n++ )
         {

-            ::EnableWindow(m_hwnds[n], enable);
+            if ( m_hwnds[n] )
+                ::EnableWindow(m_hwnds[n], enable);

         }
     }
 
         }
     }
 


@@ -105,7 +119,13 @@ public:
 
         for ( size_t n = 0; n < m_count; n++ )
         {
 
         for ( size_t n = 0; n < m_count; n++ )
         {

-            ::SendMessage(m_hwnds[n], WM_SETFONT, (WPARAM)hfont, 0);
+            if ( m_hwnds[n] )
+            {
+                ::SendMessage(m_hwnds[n], WM_SETFONT, (WPARAM)hfont, 0);
+
+                // otherwise the window might not be redrawn correctly
+                ::InvalidateRect(m_hwnds[n], NULL, FALSE /* don't erase bg */);
+            }

         }
     }
 
         }
     }
 


@@ -115,10 +135,14 @@ public:
         wxRect r;
         for ( size_t n = 0; n < m_count; n++ )
         {
         wxRect r;
         for ( size_t n = 0; n < m_count; n++ )
         {

-            RECT rc;
-            ::GetWindowRect(m_hwnds[n], &rc);
+            if ( m_hwnds[n] )
+            {
+                RECT rc;

 
 

-            r.Union(wxRectFromRECT(rc));
+                ::GetWindowRect(m_hwnds[n], &rc);
+
+                r.Union(wxRectFromRECT(rc));
+            }

         }
 
         return r;
         }
 
         return r;


@@ -137,9 +161,59 @@ private:
     // the HWNDs we contain
     HWND *m_hwnds;
 
     // the HWNDs we contain
     HWND *m_hwnds;
 

+    // the IDs of the windows
+    wxWindowIDRef *m_ids;
+

 
     DECLARE_NO_COPY_CLASS(wxSubwindows)
 };
 
 
     DECLARE_NO_COPY_CLASS(wxSubwindows)
 };
 

+// convenient macro to forward a few methods which are usually propagated to
+// subwindows to a wxSubwindows object
+//
+// parameters should be:
+//  - cname the name of the class implementing these methods
+//  - base the name of its base class
+//  - subwins the name of the member variable of type wxSubwindows *
+#define WX_FORWARD_STD_METHODS_TO_SUBWINDOWS(cname, base, subwins)            \
+    bool cname::ContainsHWND(WXHWND hWnd) const                               \
+    {                                                                         \
+        return subwins && subwins->HasWindow((HWND)hWnd);                     \
+    }                                                                         \
+                                                                              \
+    bool cname::Show(bool show)                                               \
+    {                                                                         \
+        if ( !base::Show(show) )                                              \
+            return false;                                                     \
+                                                                              \
+        if ( subwins )                                                        \
+            subwins->Show(show);                                              \
+                                                                              \
+        return true;                                                          \
+    }                                                                         \
+                                                                              \
+    bool cname::Enable(bool enable)                                           \
+    {                                                                         \
+        if ( !base::Enable(enable) )                                          \
+            return false;                                                     \
+                                                                              \
+        if ( subwins )                                                        \
+            subwins->Enable(enable);                                          \
+                                                                              \
+        return true;                                                          \
+    }                                                                         \
+                                                                              \
+    bool cname::SetFont(const wxFont& font)                                   \
+    {                                                                         \
+        if ( !base::SetFont(font) )                                           \
+            return false;                                                     \
+                                                                              \
+        if ( subwins )                                                        \
+            subwins->SetFont(font);                                           \
+                                                                              \
+        return true;                                                          \
+    }
+
+

 #endif // _WX_MSW_SUBWIN_H_
 
 #endif // _WX_MSW_SUBWIN_H_