diff options
author | Caolán McNamara <caolanm@redhat.com> | 2017-01-23 20:44:38 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2017-01-23 20:45:44 +0000 |
commit | 0b7c1dcd149cd98136e5ed3e8e0e70255d87cd16 (patch) | |
tree | ec3ca50e302eed03b07171b00b2077c1da93260c /sc | |
parent | 21297dc60514396cf3ce268726939ea5fed36654 (diff) |
Resolves: tdf#105466 crash when changing conditional formatting condition
Change-Id: Id219a7935f83acf3924e60cb1bf02311dc5c4372
Diffstat (limited to 'sc')
-rw-r--r-- | sc/source/ui/condformat/condformatdlg.cxx | 20 |
1 files changed, 12 insertions, 8 deletions
diff --git a/sc/source/ui/condformat/condformatdlg.cxx b/sc/source/ui/condformat/condformatdlg.cxx index 6cc1d1efa147..37bf0b2bb4aa 100644 --- a/sc/source/ui/condformat/condformatdlg.cxx +++ b/sc/source/ui/condformat/condformatdlg.cxx @@ -187,10 +187,12 @@ void ScCondFormatList::RecalcAll() { sal_Int32 nTotalHeight = 0; sal_Int32 nIndex = 1; - for(EntryContainer::iterator itr = maEntries.begin(); itr != maEntries.end(); ++itr) + for (const auto& item : maEntries) { - nTotalHeight += (*itr)->GetSizePixel().Height(); - (*itr)->SetIndex( nIndex ); + if (!item) + continue; + nTotalHeight += item->GetSizePixel().Height(); + item->SetIndex(nIndex); ++nIndex; } @@ -212,17 +214,19 @@ void ScCondFormatList::RecalcAll() } Point aPoint(0,-1*mpScrollBar->GetThumbPos()); - for(EntryContainer::iterator itr = maEntries.begin(); itr != maEntries.end(); ++itr) + for (const auto& item : maEntries) { - (*itr)->SetPosPixel(aPoint); - Size aSize = (*itr)->GetSizePixel(); + if (!item) + continue; + item->SetPosPixel(aPoint); + Size aSize = item->GetSizePixel(); if(mbHasScrollBar) aSize.Width() = aCtrlSize.Width() - nSrcBarSize; else aSize.Width() = aCtrlSize.Width(); - (*itr)->SetSizePixel(aSize); + item->SetSizePixel(aSize); - aPoint.Y() += (*itr)->GetSizePixel().Height(); + aPoint.Y() += item->GetSizePixel().Height(); } } |