Index: src/ch/CustomCopyDlg.cpp =================================================================== diff -u -N -r9ddf8fdd5f641491dd30c49eb90f8f740314b6af -rfadd6c9c628de875716d96c3a497b5bc6c8dca8a --- src/ch/CustomCopyDlg.cpp (.../CustomCopyDlg.cpp) (revision 9ddf8fdd5f641491dd30c49eb90f8f740314b6af) +++ src/ch/CustomCopyDlg.cpp (.../CustomCopyDlg.cpp) (revision fadd6c9c628de875716d96c3a497b5bc6c8dca8a) @@ -371,11 +371,10 @@ // refresh the entries in filters' list const chengine::TFileFiltersArray& afFilters = m_tTaskDefinition.GetFilters(); m_ctlFilters.DeleteAllItems(); - for(size_t stIndex = 0; stIndex < afFilters.GetSize(); ++stIndex) + for(size_t stIndex = 0; stIndex < afFilters.GetCount(); ++stIndex) { - const chengine::TFileFilter* pFilter = afFilters.GetAt(stIndex); - if(pFilter) - AddFilter(*pFilter, boost::numeric_cast(stIndex)); + const chengine::TFileFilter& rFilter = afFilters.GetAt(stIndex); + AddFilter(rFilter, boost::numeric_cast(stIndex)); } } @@ -539,17 +538,13 @@ CFilterDlg dlg; chengine::TFileFiltersArray& afFilters = m_tTaskDefinition.GetFilters(); - for (size_t i = 0; i < afFilters.GetSize(); i++) + for (size_t i = 0; i < afFilters.GetCount(); i++) { - const chengine::TFileFilter* pFilter = afFilters.GetAt(i); - BOOST_ASSERT(pFilter); - if(pFilter) - { - if(pFilter->GetUseMask()) - dlg.m_astrAddMask.Add(pFilter->GetCombinedMask().c_str()); - if(pFilter->GetUseExcludeMask()) - dlg.m_astrAddExcludeMask.Add(pFilter->GetCombinedExcludeMask().c_str()); - } + const chengine::TFileFilter& rFilter = afFilters.GetAt(i); + if(rFilter.GetUseMask()) + dlg.m_astrAddMask.Add(rFilter.GetCombinedMask().c_str()); + if(rFilter.GetUseExcludeMask()) + dlg.m_astrAddExcludeMask.Add(rFilter.GetCombinedExcludeMask().c_str()); } if(dlg.DoModal() == IDOK) @@ -759,22 +754,16 @@ int iItem = m_ctlFilters.GetNextSelectedItem(pos); CFilterDlg dlg; - const chengine::TFileFilter* pFilter = afFilters.GetAt(iItem); - BOOST_ASSERT(pFilter); - if(pFilter) - dlg.m_ffFilter = *pFilter; + const chengine::TFileFilter& rFilter = afFilters.GetAt(iItem); + dlg.m_ffFilter = rFilter; - for(size_t stIndex = 0; stIndex < afFilters.GetSize(); ++stIndex) + for(size_t stIndex = 0; stIndex < afFilters.GetCount(); ++stIndex) { - pFilter = afFilters.GetAt(stIndex); - BOOST_ASSERT(pFilter); - if(pFilter) - { - if(pFilter->GetUseMask() && boost::numeric_cast(stIndex) != iItem) - dlg.m_astrAddMask.Add(pFilter->GetCombinedMask().c_str()); - if (pFilter->GetUseExcludeMask() && boost::numeric_cast(stIndex) != iItem) - dlg.m_astrAddExcludeMask.Add(pFilter->GetCombinedExcludeMask().c_str()); - } + const chengine::TFileFilter& rFilter = afFilters.GetAt(stIndex); + if(rFilter.GetUseMask() && boost::numeric_cast(stIndex) != iItem) + dlg.m_astrAddMask.Add(rFilter.GetCombinedMask().c_str()); + if (rFilter.GetUseExcludeMask() && boost::numeric_cast(stIndex) != iItem) + dlg.m_astrAddExcludeMask.Add(rFilter.GetCombinedExcludeMask().c_str()); } if (dlg.DoModal() == IDOK)