diff options
author | Noel Grandin <noel@peralex.com> | 2015-10-15 12:28:31 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2015-10-15 12:29:01 +0200 |
commit | 0e8a40e8b8c883611b6d34e47dc6e33ba60e0f91 (patch) | |
tree | 92e49014bb9b368309c98d85dcfbd2f673e23d54 /chart2 | |
parent | cfc7307a23eed561152c1b016cd0ec22bc7af145 (diff) |
calling IsSet() before Call() on Link<> is unnecessary
the Call() already does a check
Found with:
git grep -A 1 -w 'IsSet()'
| grep -B 1 '.Call('
| grep ':'
| cut -d ':' -f 1
Change-Id: Ia7248f5d62640b75f705e539c3d1183e39c0d847
Diffstat (limited to 'chart2')
-rw-r--r-- | chart2/source/controller/dialogs/DataBrowser.cxx | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/chart2/source/controller/dialogs/DataBrowser.cxx b/chart2/source/controller/dialogs/DataBrowser.cxx index 84e7746ca517..00f2be300cea 100644 --- a/chart2/source/controller/dialogs/DataBrowser.cxx +++ b/chart2/source/controller/dialogs/DataBrowser.cxx @@ -226,9 +226,7 @@ SeriesHeader::~SeriesHeader() void SeriesHeader::notifyChanges() { - if( m_aChangeLink.IsSet()) - m_aChangeLink.Call( m_spSeriesName.get()); - + m_aChangeLink.Call( m_spSeriesName.get()); m_bSeriesNameChangePending = false; } @@ -745,7 +743,7 @@ void DataBrowser::CursorMoved() { EditBrowseBox::CursorMoved(); - if( GetUpdateMode() && m_aCursorMovedHdlLink.IsSet()) + if( GetUpdateMode() ) m_aCursorMovedHdlLink.Call( this ); } @@ -799,8 +797,7 @@ void DataBrowser::CellModified() { m_bDataValid = IsDataValid(); SetDirty(); - if( m_aCellModifiedLink.IsSet()) - m_aCursorMovedHdlLink.Call( this ); + m_aCursorMovedHdlLink.Call( this ); } void DataBrowser::SetDataFromModel( |