From f147b160aef1735d34c488353a7d4b875788a7e1 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Tue, 11 Feb 2020 15:56:21 +0200 Subject: clang-analyzer-deadcode.DeadStores Change-Id: Ifa384933569b27d0d08eb479bb95b799163ae386 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88450 Tested-by: Jenkins Reviewed-by: Noel Grandin --- svtools/source/svrtf/parrtf.cxx | 4 ++-- svtools/source/table/tablecontrol_impl.cxx | 12 ++++++------ 2 files changed, 8 insertions(+), 8 deletions(-) (limited to 'svtools') diff --git a/svtools/source/svrtf/parrtf.cxx b/svtools/source/svrtf/parrtf.cxx index 16d3eb7146f4..a52b8ee4f3af 100644 --- a/svtools/source/svrtf/parrtf.cxx +++ b/svtools/source/svrtf/parrtf.cxx @@ -204,7 +204,7 @@ int SvRTFParser::GetNextToken_() cAnsi = GetNextChar(); if( '\\' == cAnsi && - '\'' == ( cAnsi = GetNextChar() )) + '\'' == GetNextChar() ) // skip HexValue GetHexValue(); nNextCh = GetNextChar(); @@ -441,7 +441,7 @@ void SvRTFParser::ScanText() cAnsi = GetNextChar(); if( '\\' == cAnsi && - '\'' == ( cAnsi = GetNextChar() )) + '\'' == GetNextChar() ) // skip HexValue GetHexValue(); nNextCh = GetNextChar(); diff --git a/svtools/source/table/tablecontrol_impl.cxx b/svtools/source/table/tablecontrol_impl.cxx index a0cc092a8c04..dbdaa91ce536 100644 --- a/svtools/source/table/tablecontrol_impl.cxx +++ b/svtools/source/table/tablecontrol_impl.cxx @@ -1402,7 +1402,7 @@ namespace svt::table case cursorSelectRow: { if(m_pSelEngine->GetSelectionMode() == SelectionMode::NONE) - return bSuccess = false; + return false; //pos is the position of the current row in the vector of selected rows, if current row is selected int pos = getRowSelectedNumber(m_aSelectedRows, m_nCurRow); //if current row is selected, it should be deselected, when ALT+SPACE are pressed @@ -1423,7 +1423,7 @@ namespace svt::table case cursorSelectRowUp: { if(m_pSelEngine->GetSelectionMode() == SelectionMode::NONE) - return bSuccess = false; + return false; else if(m_pSelEngine->GetSelectionMode() == SelectionMode::Single) { //if there are other selected rows, deselect them @@ -1454,7 +1454,7 @@ namespace svt::table if(m_nCurRow>0) m_nCurRow--; else - return bSuccess = true; + return true; //if nextRow already selected, deselect it, otherwise select it if(nextRow>-1 && m_aSelectedRows[nextRow] == m_nCurRow) { @@ -1537,7 +1537,7 @@ namespace svt::table if(m_nCurRow-1 && m_aSelectedRows[nextRow] == m_nCurRow) { @@ -1618,9 +1618,9 @@ namespace svt::table case cursorSelectRowAreaBottom: { if(m_pSelEngine->GetSelectionMode() == SelectionMode::NONE) - return bSuccess = false; + return false; else if(m_pSelEngine->GetSelectionMode() == SelectionMode::Single) - return bSuccess = false; + return false; //select the region between the current and the last row RowPos iter = m_nCurRow; invalidateSelectedRegion( m_nCurRow, m_nRowCount-1 ); -- cgit