Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

a warning in the release build suppressed

Browse Source 
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@327 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Vadim Zeitlin
1998-07-23 15:56:30 +00:00
parent 14d1ccd89d
commit 3bbb630a21
1 changed files with 20 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
src/common/string.cpp
View File

@@ -258,7 +258,7 @@ void wxString::CopyBeforeWrite()
memcpy(m_pchData, pData->data(), nLen*sizeof(char)); memcpy(m_pchData, pData->data(), nLen*sizeof(char));
} }
wxASSERT( !pData->IsShared() ); // we must be the only owner wxASSERT( !GetStringData()->IsShared() ); // we must be the only owner
} }
// must be called before replacing contents of this string // must be called before replacing contents of this string
@@ -323,8 +323,14 @@ void wxString::Alloc(uint nLen)
void wxString::Shrink() void wxString::Shrink()
{ {
wxStringData *pData = GetStringData(); wxStringData *pData = GetStringData();
void *p = realloc(pData, sizeof(wxStringData) +
(pData->nDataLength + 1)*sizeof(char)); // this variable is unused in release build, so avoid the compiler warning by
// just not declaring it
#ifdef __WXDEBUG__
void *p =
#endif
realloc(pData, sizeof(wxStringData) + (pData->nDataLength + 1)*sizeof(char));
wxASSERT( p != NULL ); // can't free memory? wxASSERT( p != NULL ); // can't free memory?
wxASSERT( p == pData ); // we're decrementing the size - block shouldn't move! wxASSERT( p == pData ); // we're decrementing the size - block shouldn't move!
} }
@@ -1073,6 +1079,8 @@ wxArrayString::wxArrayString()
// copy ctor // copy ctor
wxArrayString::wxArrayString(const wxArrayString& src) wxArrayString::wxArrayString(const wxArrayString& src)
{ {
m_nSize =
m_nCount = 0;
m_pItems = NULL; m_pItems = NULL;
*this = src; *this = src;
@@ -1081,7 +1089,7 @@ wxArrayString::wxArrayString(const wxArrayString& src)
// assignment operator // assignment operator
wxArrayString& wxArrayString::operator=(const wxArrayString& src) wxArrayString& wxArrayString::operator=(const wxArrayString& src)
{ {
DELETEA(m_pItems); Clear();
m_nSize = 0; m_nSize = 0;
if ( src.m_nCount > ARRAY_DEFAULT_INITIAL_SIZE ) if ( src.m_nCount > ARRAY_DEFAULT_INITIAL_SIZE )
@@ -1092,6 +1100,9 @@ wxArrayString& wxArrayString::operator=(const wxArrayString& src)
for ( uint n = 0; n < src.m_nCount; n++ ) for ( uint n = 0; n < src.m_nCount; n++ )
Add(src[n]); Add(src[n]);
if ( m_nCount != 0 )
memcpy(m_pItems, src.m_pItems, m_nCount*sizeof(char *));
return *this; return *this;
} }
@@ -1106,6 +1117,10 @@ void wxArrayString::Grow()
m_pItems = new char *[m_nSize]; m_pItems = new char *[m_nSize];
} }
else { else {
// otherwise when it's called for the first time, nIncrement would be 0
// and the array would never be expanded
wxASSERT( ARRAY_DEFAULT_INITIAL_SIZE != 0 );
// add 50% but not too much // add 50% but not too much
size_t nIncrement = m_nSize < ARRAY_DEFAULT_INITIAL_SIZE size_t nIncrement = m_nSize < ARRAY_DEFAULT_INITIAL_SIZE
? ARRAY_DEFAULT_INITIAL_SIZE : m_nSize >> 1; ? ARRAY_DEFAULT_INITIAL_SIZE : m_nSize >> 1;