Index: src/libchcore/ConfigNodeContainer.cpp =================================================================== diff -u -N -rf354b6f39d2a0425c1827906c84cbc1377b8c8d3 -ra44714d5c7ec0f50a376f4d0ea919ee5a224f834 --- src/libchcore/ConfigNodeContainer.cpp (.../ConfigNodeContainer.cpp) (revision f354b6f39d2a0425c1827906c84cbc1377b8c8d3) +++ src/libchcore/ConfigNodeContainer.cpp (.../ConfigNodeContainer.cpp) (revision a44714d5c7ec0f50a376f4d0ea919ee5a224f834) @@ -75,7 +75,7 @@ /////////////////////////////////////////////////////////////////////// ConfigNodeContainer::ConfigNodeContainer() : m_bDelayedEnabled(false), - m_stLastObjectID(0) + m_oidLastObjectID(0) { } @@ -86,7 +86,7 @@ m_strFilePath = rSrc.m_strFilePath; m_setDelayedNotifications.Clear(); m_bDelayedEnabled = false; - m_stLastObjectID = rSrc.m_stLastObjectID; + m_oidLastObjectID = rSrc.m_oidLastObjectID; } ConfigNodeContainer& ConfigNodeContainer::operator=(const ConfigNodeContainer& rSrc) @@ -99,7 +99,7 @@ m_mic = rSrc.m_mic; m_strFilePath = rSrc.m_strFilePath; m_bDelayedEnabled = false; - m_stLastObjectID = rSrc.m_stLastObjectID; + m_oidLastObjectID = rSrc.m_oidLastObjectID; m_setDelayedNotifications.Clear(); } @@ -163,7 +163,7 @@ // insert new items for(size_t stIndex = 0; stIndex < rValue.GetCount(); ++stIndex) { - m_mic.insert(ConfigNode(++m_stLastObjectID, pszPropName, boost::numeric_cast(stIndex), rValue.GetAt(stIndex))); + m_mic.insert(ConfigNode(++m_oidLastObjectID, pszPropName, boost::numeric_cast(stIndex), rValue.GetAt(stIndex))); } return false; @@ -186,7 +186,7 @@ else { // delete this item - m_setRemovedObjects.Add(pairFnd.first->m_stObjectID); + m_setRemovedObjects.Add(pairFnd.first->m_oidObjectID); pairFnd.first = m_mic.erase(pairFnd.first); } @@ -196,7 +196,7 @@ while(stIndex < rValue.GetCount()) { // add items not added before (with new oids) - m_mic.insert(ConfigNode(++m_stLastObjectID, pszPropName, boost::numeric_cast(stIndex), rValue.GetAt(stIndex))); + m_mic.insert(ConfigNode(++m_oidLastObjectID, pszPropName, boost::numeric_cast(stIndex), rValue.GetAt(stIndex))); ++stIndex; } @@ -220,7 +220,7 @@ { if(rNode.m_strNodeName.Get().StartsWith(m_strPrefix.c_str())) { - m_rRemovedObjects.Add(rNode.m_stObjectID); + m_rRemovedObjects.Add(rNode.m_oidObjectID); return true; } return false; @@ -271,7 +271,7 @@ TString strName = iter->m_strNodeName.Get(); strName.MidSelf(strReplace.GetLength()); - tNewContainer.m_mic.insert(ConfigNode(++tNewContainer.m_stLastObjectID, strName, iter->GetOrder(), iter->m_strValue)); + tNewContainer.m_mic.insert(ConfigNode(++tNewContainer.m_oidLastObjectID, strName, iter->GetOrder(), iter->m_strValue)); } } @@ -313,7 +313,7 @@ } strName.Delete(0, stPos + 2); // skip "]." at the beginning - pCurrentContainer->m_mic.insert(ConfigNode(++pCurrentContainer->m_stLastObjectID, strName, iter->GetOrder(), iter->m_strValue)); + pCurrentContainer->m_mic.insert(ConfigNode(++pCurrentContainer->m_oidLastObjectID, strName, iter->GetOrder(), iter->m_strValue)); } } @@ -357,15 +357,15 @@ else { // node does not exist - need to add new one - m_mic.insert(ConfigNode(++m_stLastObjectID, strNodeName, iter->GetOrder(), iter->m_strValue)); + m_mic.insert(ConfigNode(++m_oidLastObjectID, strNodeName, iter->GetOrder(), iter->m_strValue)); } // remove all nodes with names from setExisting BOOST_FOREACH(const PairInfo& pairNode, setExistingNames) { NodeContainer::iterator iterToRemove = m_mic.find(boost::make_tuple(pairNode.first, pairNode.second)); if(iterToRemove != m_mic.end()) - m_setRemovedObjects.Add(iterToRemove->m_stObjectID); + m_setRemovedObjects.Add(iterToRemove->m_oidObjectID); m_mic.erase(iterToRemove); } @@ -406,7 +406,7 @@ { TString strNodeName = strNodePrefix + iter->m_strNodeName; - m_mic.insert(ConfigNode(++m_stLastObjectID, strNodeName, iter->GetOrder(), iter->m_strValue)); + m_mic.insert(ConfigNode(++m_oidLastObjectID, strNodeName, iter->GetOrder(), iter->m_strValue)); } } @@ -461,7 +461,7 @@ case eMode_LeafStringArrayEntries: { strNewPath = strCurrentPath + rNode.first.c_str(); - m_mic.insert(ConfigNode(++m_stLastObjectID, strNewPath, iIndex++, rNode.second.get_value().c_str())); + m_mic.insert(ConfigNode(++m_oidLastObjectID, strNewPath, iIndex++, rNode.second.get_value().c_str())); break; } case eMode_LeafOrContainer: @@ -470,7 +470,7 @@ if(rNode.second.empty()) { // get leaf info - m_mic.insert(ConfigNode(++m_stLastObjectID, strNewPath, 0, rNode.second.get_value().c_str())); + m_mic.insert(ConfigNode(++m_oidLastObjectID, strNewPath, 0, rNode.second.get_value().c_str())); } else { @@ -501,7 +501,9 @@ // iterate through property tree ImportNode(_T(""), rTree); +#ifdef _DEBUG Dump(); +#endif } void ConfigNodeContainer::ExportToPropertyTree(boost::property_tree::wiptree& rTree) const @@ -569,21 +571,24 @@ rTree.add_child(strGroupNode.c_str(), treeSubnodes); } +#ifdef _DEBUG void ConfigNodeContainer::Dump() { const size_t stBufferSize = 1024; TCHAR szBuffer[stBufferSize]; for(NodeContainer::const_iterator iter = m_mic.begin(); iter != m_mic.end(); ++iter) { - _sntprintf_s(szBuffer, stBufferSize, _TRUNCATE, _T("Node (oid %Iu): %s.%ld = %s\n"), iter->m_stObjectID, iter->m_strNodeName.Get().c_str(), iter->m_iOrder.Get(), iter->m_strValue.Get().c_str()); + unsigned long long ullID = iter->m_oidObjectID; + _sntprintf_s(szBuffer, stBufferSize, _TRUNCATE, _T("Node (oid %I64u): %s.%ld = %s\n"), ullID, iter->m_strNodeName.Get().c_str(), iter->m_iOrder.Get(), iter->m_strValue.Get().c_str()); OutputDebugString(szBuffer); } } +#endif void ConfigNodeContainer::AddEntry(PCTSTR pszPropName, int iIndex, const TString& strValue) { - std::pair pairInsert = m_mic.insert(ConfigNode(++m_stLastObjectID, pszPropName, iIndex, strValue)); + std::pair pairInsert = m_mic.insert(ConfigNode(++m_oidLastObjectID, pszPropName, iIndex, strValue)); pairInsert.first->m_setModifications.reset(); } }