fixing reentrancy which happened in tests, bringing client coordinates origins in...

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@74197 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

diff --git a/src/generic/listctrl.cpp b/src/generic/listctrl.cpp
index 54f7f0dd5740b88a01c0b1c6da27e087e03df017..bf9448a331714e43d7079d9d8311e479a4ccf97c 100644 (file)
--- a/src/generic/listctrl.cpp
+++ b/src/generic/listctrl.cpp
@@ -4509,6 +4509,17 @@ void wxGenericListCtrl::CreateOrDestroyHeaderWindowAsNeeded()
 
     if (needs_header)
     {
 
     if (needs_header)
     {

+        // since there is no separate Create method for the wxListHeaderWindow
+        // we have to guard against reentrancy which happens via new wxListHeaderWindow ->
+        // wxNavigationEnabled::AddChild -> ToggleWindowStyle -> SetWindowStyleFlag
+        // since has_header is still false then
+        static bool blockreentrancy = false;
+        
+        if ( blockreentrancy )
+            return;
+
+        blockreentrancy = true;
+

         m_headerWin = new wxListHeaderWindow
                       (
                         this, wxID_ANY, m_mainWin,
         m_headerWin = new wxListHeaderWindow
                       (
                         this, wxID_ANY, m_mainWin,


@@ -4520,7 +4531,8 @@ void wxGenericListCtrl::CreateOrDestroyHeaderWindowAsNeeded()
                         ),
                         wxTAB_TRAVERSAL
                       );
                         ),
                         wxTAB_TRAVERSAL
                       );

-
+        blockreentrancy = false;
+        

 #if defined( __WXMAC__ )
         static wxFont font( wxOSX_SYSTEM_FONT_SMALL );
         m_headerWin->SetFont( font );
 #if defined( __WXMAC__ )
         static wxFont font( wxOSX_SYSTEM_FONT_SMALL );
         m_headerWin->SetFont( font );


@@ -5291,21 +5303,29 @@ bool wxGenericListCtrl::DoPopupMenu( wxMenu *menu, int x, int y )
 
 void wxGenericListCtrl::DoClientToScreen( int *x, int *y ) const
 {
 
 void wxGenericListCtrl::DoClientToScreen( int *x, int *y ) const
 {

+    // having (0,0) at the origin of the m_mainWin seems wrong compared to
+    // the other code like in Refresh
+#if 0

     // It's not clear whether this can be called before m_mainWin is created
     // but it seems better to be on the safe side and check.
     if ( m_mainWin )
         m_mainWin->DoClientToScreen(x, y);
     else
     // It's not clear whether this can be called before m_mainWin is created
     // but it seems better to be on the safe side and check.
     if ( m_mainWin )
         m_mainWin->DoClientToScreen(x, y);
     else

+#endif

         wxListCtrlBase::DoClientToScreen(x, y);
 }
 
 void wxGenericListCtrl::DoScreenToClient( int *x, int *y ) const
 {
         wxListCtrlBase::DoClientToScreen(x, y);
 }
 
 void wxGenericListCtrl::DoScreenToClient( int *x, int *y ) const
 {

+    // having (0,0) at the origin of the m_mainWin seems wrong compared to
+    // the other code like in Refresh
+#if 0

     // At least in wxGTK/Univ build this method can be called before m_mainWin
     // is created so avoid crashes in this case.
     if ( m_mainWin )
         m_mainWin->DoScreenToClient(x, y);
     else
     // At least in wxGTK/Univ build this method can be called before m_mainWin
     // is created so avoid crashes in this case.
     if ( m_mainWin )
         m_mainWin->DoScreenToClient(x, y);
     else

+#endif

         wxListCtrlBase::DoScreenToClient(x, y);
 }
 
         wxListCtrlBase::DoScreenToClient(x, y);
 }