Reuse wxMessageOutputStderr for wxLogStderr implementation.

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

diff --git a/include/wx/msw/subwin.h b/include/wx/msw/subwin.h
index cbe712e73c61010d71a6b1ae3058fb0eab54561d..90be4841a1883a6bb82a34fa86f1159ed826ea9f 100644 (file)
--- a/include/wx/msw/subwin.h
+++ b/include/wx/msw/subwin.h
@@ -18,7 +18,7 @@
 // wxSubwindows contains all HWNDs making part of a single wx control
 // ----------------------------------------------------------------------------
 
 // wxSubwindows contains all HWNDs making part of a single wx control
 // ----------------------------------------------------------------------------
 

-class WXDLLEXPORT wxSubwindows
+class WXDLLIMPEXP_CORE wxSubwindows

 {
 public:
     // the number of subwindows can be specified either as parameter to ctor or
 {
 public:
     // the number of subwindows can be specified either as parameter to ctor or


@@ -28,7 +28,7 @@ public:
     // allocate enough space for the given number of windows
     void Create(size_t n)
     {
     // allocate enough space for the given number of windows
     void Create(size_t n)
     {

-        wxASSERT_MSG( !m_hwnds, _T("Create() called twice?") );
+        wxASSERT_MSG( !m_hwnds, wxT("Create() called twice?") );

 
         m_count = n;
         m_hwnds = (HWND *)calloc(n, sizeof(HWND));
 
         m_count = n;
         m_hwnds = (HWND *)calloc(n, sizeof(HWND));


@@ -40,7 +40,8 @@ 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);


@@ -53,21 +54,21 @@ public:
     // access a given window
     HWND& Get(size_t n)
     {
     // access a given window
     HWND& Get(size_t n)
     {

-        wxASSERT_MSG( n < m_count, _T("subwindow index out of range") );
+        wxASSERT_MSG( n < m_count, wxT("subwindow index out of range") );

 
         return m_hwnds[n];
     }
 
     HWND operator[](size_t n) const
     {
 
         return m_hwnds[n];
     }
 
     HWND operator[](size_t n) const
     {

-        return wx_const_cast(wxSubwindows *, this)->Get(n);
+        return 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)
     {
     }
 
     // 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") );
+        wxASSERT_MSG( n < m_count, wxT("subwindow index out of range") );

 
         m_hwnds[n] = hwnd;
         m_ids[n] = id;
 
         m_hwnds[n] = hwnd;
         m_ids[n] = id;


@@ -95,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);

         }
     }
 
         }
     }
 


@@ -104,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);

         }
     }
 
         }
     }
 


@@ -112,14 +115,17 @@ public:
     void SetFont(const wxFont& font)
     {
         HFONT hfont = GetHfontOf(font);
     void SetFont(const wxFont& font)
     {
         HFONT hfont = GetHfontOf(font);

-        wxCHECK_RET( hfont, _T("invalid font") );
+        wxCHECK_RET( hfont, wxT("invalid font") );

 
         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 */);
+                // otherwise the window might not be redrawn correctly
+                ::InvalidateRect(m_hwnds[n], NULL, FALSE /* don't erase bg */);
+            }

         }
     }
 
         }
     }
 


@@ -129,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;


@@ -155,7 +165,7 @@ private:
     wxWindowIDRef *m_ids;
 
 
     wxWindowIDRef *m_ids;
 
 

-    DECLARE_NO_COPY_CLASS(wxSubwindows)
+    wxDECLARE_NO_COPY_CLASS(wxSubwindows);

 };
 
 // convenient macro to forward a few methods which are usually propagated to
 };
 
 // convenient macro to forward a few methods which are usually propagated to