X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/a121d7205216f5b4b144264096e07104b5ad3172..78743282716f48121aa17cc76068bfdfbfc4e9bc:/src/common/fileconf.cpp diff --git a/src/common/fileconf.cpp b/src/common/fileconf.cpp index 5e445db8ce..afb7f15282 100644 --- a/src/common/fileconf.cpp +++ b/src/common/fileconf.cpp @@ -49,6 +49,7 @@ #if defined(__WXMAC__) #include "wx/mac/private.h" // includes mac headers + #include "wx/filename.h" // for MacSetTypeAndCreator #endif #if defined(__WXMSW__) @@ -75,6 +76,8 @@ #define MAX_PATH 512 #endif +#define FILECONF_TRACE_MASK _T("fileconf") + // ---------------------------------------------------------------------------- // global functions declarations // ---------------------------------------------------------------------------- @@ -203,6 +206,9 @@ private: // DeleteSubgroupByName helper bool DeleteSubgroup(wxFileConfigGroup *pGroup); + // used by Rename() + void UpdateGroupAndSubgroupsLines(); + public: // ctor wxFileConfigGroup(wxFileConfigGroup *pParent, const wxString& strName, wxFileConfig *); @@ -527,7 +533,8 @@ wxFileConfig::wxFileConfig(wxInputStream &inStream, wxMBConv& conv) } // also add whatever we have left in the translated string. - memText.AddLine(strTrans); + if ( !strTrans.empty() ) + memText.AddLine(strTrans); // Finally we can parse it all. Parse(memText, true /* local */); @@ -890,7 +897,7 @@ bool wxFileConfig::DoWriteString(const wxString& key, const wxString& szValue) wxConfigPathChanger path(this, key); wxString strName = path.Name(); - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" Writing String '%s' = '%s' to Group '%s'"), strName.c_str(), szValue.c_str(), @@ -904,7 +911,7 @@ bool wxFileConfig::DoWriteString(const wxString& key, const wxString& szValue) // ... except if it's empty in which case it's a way to force it's creation - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" Creating group %s"), m_pCurrentGroup->Name().c_str() ); @@ -928,13 +935,13 @@ bool wxFileConfig::DoWriteString(const wxString& key, const wxString& szValue) if ( pEntry == 0 ) { - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" Adding Entry %s"), strName.c_str() ); pEntry = m_pCurrentGroup->AddEntry(strName); } - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" Setting value %s"), szValue.c_str() ); pEntry->SetValue(szValue); @@ -1025,6 +1032,9 @@ bool wxFileConfig::Save(wxOutputStream& os, wxMBConv& conv) bool wxFileConfig::RenameEntry(const wxString& oldName, const wxString& newName) { + wxASSERT_MSG( !wxStrchr(oldName, wxCONFIG_PATH_SEPARATOR), + _T("RenameEntry(): paths are not supported") ); + // check that the entry exists wxFileConfigEntry *oldEntry = m_pCurrentGroup->FindEntry(oldName); if ( !oldEntry ) @@ -1106,13 +1116,19 @@ bool wxFileConfig::DeleteAll() { CleanUp(); - if ( wxFile::Exists(m_strLocalFile) && wxRemove(m_strLocalFile) == -1 ) + if ( !m_strLocalFile.empty() ) { - wxLogSysError(_("can't delete user configuration file '%s'"), m_strLocalFile.c_str()); - return false; + if ( wxFile::Exists(m_strLocalFile) && wxRemove(m_strLocalFile) == -1 ) + { + wxLogSysError(_("can't delete user configuration file '%s'"), + m_strLocalFile.c_str()); + return false; + } + + m_strLocalFile = + m_strGlobalFile = wxT(""); } - m_strLocalFile = m_strGlobalFile = wxT(""); Init(); return true; @@ -1126,13 +1142,13 @@ bool wxFileConfig::DeleteAll() wxFileConfigLineList *wxFileConfig::LineListAppend(const wxString& str) { - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" ** Adding Line '%s'"), str.c_str() ); - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" head: %s"), ((m_linesHead) ? m_linesHead->Text().c_str() : wxEmptyString) ); - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" tail: %s"), ((m_linesTail) ? m_linesTail->Text().c_str() : wxEmptyString) ); @@ -1152,10 +1168,10 @@ wxFileConfigLineList *wxFileConfig::LineListAppend(const wxString& str) m_linesTail = pLine; - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" head: %s"), ((m_linesHead) ? m_linesHead->Text().c_str() : wxEmptyString) ); - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" tail: %s"), ((m_linesTail) ? m_linesTail->Text().c_str() : wxEmptyString) ); @@ -1166,14 +1182,14 @@ wxFileConfigLineList *wxFileConfig::LineListAppend(const wxString& str) wxFileConfigLineList *wxFileConfig::LineListInsert(const wxString& str, wxFileConfigLineList *pLine) { - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" ** Inserting Line '%s' after '%s'"), str.c_str(), ((pLine) ? pLine->Text().c_str() : wxEmptyString) ); - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" head: %s"), ((m_linesHead) ? m_linesHead->Text().c_str() : wxEmptyString) ); - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" tail: %s"), ((m_linesTail) ? m_linesTail->Text().c_str() : wxEmptyString) ); @@ -1198,10 +1214,10 @@ wxFileConfigLineList *wxFileConfig::LineListInsert(const wxString& str, pLine->SetNext(pNewLine); } - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" head: %s"), ((m_linesHead) ? m_linesHead->Text().c_str() : wxEmptyString) ); - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" tail: %s"), ((m_linesTail) ? m_linesTail->Text().c_str() : wxEmptyString) ); @@ -1210,13 +1226,13 @@ wxFileConfigLineList *wxFileConfig::LineListInsert(const wxString& str, void wxFileConfig::LineListRemove(wxFileConfigLineList *pLine) { - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" ** Removing Line '%s'"), pLine->Text().c_str() ); - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" head: %s"), ((m_linesHead) ? m_linesHead->Text().c_str() : wxEmptyString) ); - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" tail: %s"), ((m_linesTail) ? m_linesTail->Text().c_str() : wxEmptyString) ); @@ -1237,10 +1253,10 @@ void wxFileConfig::LineListRemove(wxFileConfigLineList *pLine) else pNext->SetPrev(pPrev); - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" head: %s"), ((m_linesHead) ? m_linesHead->Text().c_str() : wxEmptyString) ); - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" tail: %s"), ((m_linesTail) ? m_linesTail->Text().c_str() : wxEmptyString) ); @@ -1336,13 +1352,13 @@ void wxFileConfigGroup::SetLine(wxFileConfigLineList *pLine) // have it or in the very beginning if we're the root group. wxFileConfigLineList *wxFileConfigGroup::GetGroupLine() { - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" GetGroupLine() for Group '%s'"), Name().c_str() ); if ( !m_pLine ) { - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" Getting Line item pointer") ); wxFileConfigGroup *pParent = Parent(); @@ -1350,7 +1366,7 @@ wxFileConfigLineList *wxFileConfigGroup::GetGroupLine() // this group wasn't present in local config file, add it now if ( pParent ) { - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" checking parent '%s'"), pParent->Name().c_str() ); @@ -1396,7 +1412,7 @@ wxFileConfigLineList *wxFileConfigGroup::GetLastGroupLine() // one immediately after the group line itself. wxFileConfigLineList *wxFileConfigGroup::GetLastEntryLine() { - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" GetLastEntryLine() for Group '%s'"), Name().c_str() ); @@ -1433,28 +1449,41 @@ void wxFileConfigGroup::SetLastEntry(wxFileConfigEntry *pEntry) // group name // ---------------------------------------------------------------------------- +void wxFileConfigGroup::UpdateGroupAndSubgroupsLines() +{ + // update the line of this group + wxFileConfigLineList *line = GetGroupLine(); + wxCHECK_RET( line, _T("a non root group must have a corresponding line!") ); + + // +1: skip the leading '/' + line->SetText(wxString::Format(_T("[%s]"), GetFullName().c_str() + 1)); + + + // also update all subgroups as they have this groups name in their lines + const size_t nCount = m_aSubgroups.Count(); + for ( size_t n = 0; n < nCount; n++ ) + { + m_aSubgroups[n]->UpdateGroupAndSubgroupsLines(); + } +} + void wxFileConfigGroup::Rename(const wxString& newName) { wxCHECK_RET( m_pParent, _T("the root group can't be renamed") ); m_strName = newName; - // +1: no leading '/' - wxString strFullName; - strFullName << wxT("[") << (GetFullName().c_str() + 1) << wxT("]"); - - wxFileConfigLineList *line = GetGroupLine(); - wxCHECK_RET( line, _T("a non root group must have a corresponding line!") ); - - line->SetText(strFullName); + // update the group lines recursively + UpdateGroupAndSubgroupsLines(); } wxString wxFileConfigGroup::GetFullName() const { - if ( Parent() ) - return Parent()->GetFullName() + wxCONFIG_PATH_SEPARATOR + Name(); - else - return wxT(""); + wxString fullname; + if ( Parent() ) + fullname = Parent()->GetFullName() + wxCONFIG_PATH_SEPARATOR + Name(); + + return fullname; } // ---------------------------------------------------------------------------- @@ -1572,121 +1601,91 @@ bool wxFileConfigGroup::DeleteSubgroup(wxFileConfigGroup *pGroup) { wxCHECK_MSG( pGroup, false, _T("deleting non existing group?") ); - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T("Deleting group '%s' from '%s'"), pGroup->Name().c_str(), Name().c_str() ); - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" (m_pLine) = prev: %p, this %p, next %p"), ((m_pLine) ? m_pLine->Prev() : 0), m_pLine, ((m_pLine) ? m_pLine->Next() : 0) ); - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" text: '%s'"), ((m_pLine) ? m_pLine->Text().c_str() : wxEmptyString) ); - // delete all entries + // delete all entries... size_t nCount = pGroup->m_aEntries.Count(); - wxLogTrace(_T("wxFileConfig"), - _T("Removing %lu Entries"), - (unsigned long)nCount ); + wxLogTrace(FILECONF_TRACE_MASK, + _T("Removing %lu entries"), (unsigned long)nCount ); for ( size_t nEntry = 0; nEntry < nCount; nEntry++ ) { - wxFileConfigLineList *pLine = pGroup->m_aEntries[nEntry]->GetLine(); + wxFileConfigLineList *pLine = pGroup->m_aEntries[nEntry]->GetLine(); - if ( pLine != 0 ) + if ( pLine ) { - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" '%s'"), pLine->Text().c_str() ); m_pConfig->LineListRemove(pLine); } } - // and subgroups of this subgroup - + // ...and subgroups of this subgroup nCount = pGroup->m_aSubgroups.Count(); - wxLogTrace( _T("wxFileConfig"), - _T("Removing %lu SubGroups"), - (unsigned long)nCount ); + wxLogTrace( FILECONF_TRACE_MASK, + _T("Removing %lu subgroups"), (unsigned long)nCount ); for ( size_t nGroup = 0; nGroup < nCount; nGroup++ ) { pGroup->DeleteSubgroup(pGroup->m_aSubgroups[0]); } - // finally the group itself - - wxFileConfigLineList *pLine = pGroup->m_pLine; - - if ( pLine != 0 ) + // and then finally the group itself + wxFileConfigLineList *pLine = pGroup->m_pLine; + if ( pLine ) { - wxLogTrace( _T("wxFileConfig"), - _T(" Removing line entry for Group '%s' : '%s'"), + wxLogTrace( FILECONF_TRACE_MASK, + _T(" Removing line for group '%s' : '%s'"), pGroup->Name().c_str(), pLine->Text().c_str() ); - wxLogTrace( _T("wxFileConfig"), - _T(" Removing from Group '%s' : '%s'"), + wxLogTrace( FILECONF_TRACE_MASK, + _T(" Removing from group '%s' : '%s'"), Name().c_str(), ((m_pLine) ? m_pLine->Text().c_str() : wxEmptyString) ); - // notice that we may do this test inside the previous "if" - // because the last entry's line is surely !NULL - + // notice that we may do this test inside the previous "if" + // because the last entry's line is surely !NULL if ( pGroup == m_pLastGroup ) { - wxLogTrace( _T("wxFileConfig"), - _T(" ------- Removing last group -------") ); - - // our last entry is being deleted, so find the last one which stays. - // go back until we find a subgroup or reach the group's line, unless - // we are the root group, which we'll notice shortly. - - wxFileConfigGroup *pNewLast = 0; - size_t nSubgroups = m_aSubgroups.Count(); - wxFileConfigLineList *pl; - - for ( pl = pLine->Prev(); pl != m_pLine; pl = pl->Prev() ) + wxLogTrace( FILECONF_TRACE_MASK, + _T(" Removing last group") ); + + // our last entry is being deleted, so find the last one which + // stays by going back until we find a subgroup or reach the + // group line + const size_t nSubgroups = m_aSubgroups.Count(); + + m_pLastGroup = NULL; + for ( wxFileConfigLineList *pl = pLine->Prev(); + pl && pl != m_pLine && !m_pLastGroup; + pl = pl->Prev() ) { - // is it our subgroup? - - for ( size_t n = 0; (pNewLast == 0) && (n < nSubgroups); n++ ) + // does this line belong to our subgroup? + for ( size_t n = 0; n < nSubgroups; n++ ) { - // do _not_ call GetGroupLine! we don't want to add it to the local - // file if it's not already there - - if ( m_aSubgroups[n]->m_pLine == m_pLine ) - pNewLast = m_aSubgroups[n]; + // do _not_ call GetGroupLine! we don't want to add it to + // the local file if it's not already there + if ( m_aSubgroups[n]->m_pLine == pl ) + { + m_pLastGroup = m_aSubgroups[n]; + break; + } } - - if ( pNewLast != 0 ) // found? - break; - } - - if ( pl == m_pLine || m_pParent == 0 ) - { - wxLogTrace( _T("wxFileConfig"), - _T(" ------- No previous group found -------") ); - - wxASSERT_MSG( !pNewLast || m_pLine == 0, - _T("how comes it has the same line as we?") ); - - // we've reached the group line without finding any subgroups, - // or realised we removed the last group from the root. - - m_pLastGroup = 0; - } - else - { - wxLogTrace( _T("wxFileConfig"), - _T(" ------- Last Group set to '%s' -------"), - pNewLast->Name().c_str() ); - - m_pLastGroup = pNewLast; } } @@ -1694,7 +1693,7 @@ bool wxFileConfigGroup::DeleteSubgroup(wxFileConfigGroup *pGroup) } else { - wxLogTrace( _T("wxFileConfig"), + wxLogTrace( FILECONF_TRACE_MASK, _T(" No line entry for Group '%s'?"), pGroup->Name().c_str() ); } @@ -1708,7 +1707,11 @@ bool wxFileConfigGroup::DeleteSubgroup(wxFileConfigGroup *pGroup) bool wxFileConfigGroup::DeleteEntry(const wxChar *szName) { wxFileConfigEntry *pEntry = FindEntry(szName); - wxCHECK( pEntry != NULL, false ); // deleting non existing item? + if ( !pEntry ) + { + // entry doesn't exist, nothing to do + return false; + } wxFileConfigLineList *pLine = pEntry->GetLine(); if ( pLine != NULL ) { @@ -1833,8 +1836,20 @@ void wxFileConfigEntry::SetValue(const wxString& strValue, bool bUser) } else // this entry didn't exist in the local file { - // add a new line to the file + // add a new line to the file: note the hack for the root group + // which is special in that it doesn't have its own group line + // (something like "[/]") and so the line we get for it may be not + // its line at all if it doesn't have any entries + // + // this is definitely not the right place to fix it but changing + // the root group to have NULL m_pLine will probably break too + // much stuff elsewhere so I don't dare to do it... wxFileConfigLineList *line = Group()->GetLastEntryLine(); + if ( !Group()->Parent() && line == Group()->GetGroupLine() ) + { + // prepend the first root group entry to the head of the list + line = NULL; + } m_pLine = Group()->Config()->LineListInsert(strLine, line); Group()->SetLastEntry(this); @@ -2032,5 +2047,3 @@ static wxString GetAppName(const wxString& appName) #endif // wxUSE_CONFIG - -// vi:sts=4:sw=4:et