diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-11-16 14:06:06 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-11-16 15:05:58 +0100 |
| commit | c8907b19dc04a57b74cfc14dc641ac41177e2142 (patch) | |
| tree | abd7d18e2faa6f995144d40a72e06a04eae034ac /sw/source/core/doc/DocumentRedlineManager.cxx | |
| parent | 11300c48d4350c2f6748a1ca04b919cfae338d41 (diff) | |
loplugin:buriedassign in sw
Change-Id: If2adf22a0ac3e030fca1b4ecd0173cac58f0f21e
Reviewed-on: https://gerrit.libreoffice.org/63470
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sw/source/core/doc/DocumentRedlineManager.cxx')
| -rw-r--r-- | sw/source/core/doc/DocumentRedlineManager.cxx | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/sw/source/core/doc/DocumentRedlineManager.cxx b/sw/source/core/doc/DocumentRedlineManager.cxx index a5b60561abdf..036e33e43853 100644 --- a/sw/source/core/doc/DocumentRedlineManager.cxx +++ b/sw/source/core/doc/DocumentRedlineManager.cxx @@ -2342,7 +2342,10 @@ bool DocumentRedlineManager::AcceptRedline( SwRedlineTable::size_type nPos, bool if( SwRedlineTable::npos != nFndPos || ( 0 != ( --nLoopCnt ) && SwRedlineTable::npos != ( nFndPos = mpRedlineTable->FindPrevSeqNo( nSeqNo, nPos ))) ) - pTmp = (*mpRedlineTable)[ nPos = nFndPos ]; + { + nPos = nFndPos; + pTmp = (*mpRedlineTable)[ nPos ]; + } else nLoopCnt = 0; } @@ -2481,7 +2484,10 @@ bool DocumentRedlineManager::RejectRedline( SwRedlineTable::size_type nPos, bool if( SwRedlineTable::npos != nFndPos || ( 0 != ( --nLoopCnt ) && SwRedlineTable::npos != ( nFndPos = mpRedlineTable->FindPrevSeqNo( nSeqNo, nPos ))) ) - pTmp = (*mpRedlineTable)[ nPos = nFndPos ]; + { + nPos = nFndPos; + pTmp = (*mpRedlineTable)[ nPos ]; + } else nLoopCnt = 0; } |