From: Julian Smart <julian@anthemion.co.uk>
Date: Fri, 10 Dec 2004 11:24:10 +0000 (+0000)
Subject: Workaround for memory bug when using wxRegConfig and calling
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/b9a59c919c8f76d8e5187258ce68ab3034316872

Workaround for memory bug when using wxRegConfig and calling
SetPath.


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

diff --git a/src/common/config.cpp b/src/common/config.cpp
index 30ca1fa2e7..f2d95677a4 100644
--- a/src/common/config.cpp
+++ b/src/common/config.cpp
@@ -246,7 +246,19 @@ wxConfigPathChanger::wxConfigPathChanger(const wxConfigBase *pContainer,
     {
         // do change the path
         m_bChanged = true;
-        m_strOldPath = m_pContainer->GetPath();
+
+        /* JACS: work around a memory bug that causes an assert
+           when using wxRegConfig, related to reference-counting.
+           Can be reproduced by removing (const wxChar*) below and
+           adding the following code to the config sample OnInit under
+           Windows:
+
+           pConfig->SetPath(wxT("MySettings"));
+           pConfig->SetPath(wxT(".."));
+           int value;
+           pConfig->Read(_T("MainWindowX"), & value);
+        */
+        m_strOldPath = (const wxChar*) m_pContainer->GetPath();
         if ( *m_strOldPath.c_str() != wxCONFIG_PATH_SEPARATOR )
           m_strOldPath += wxCONFIG_PATH_SEPARATOR;
         m_pContainer->SetPath(strPath);