diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2014-10-01 16:51:24 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2014-10-01 20:29:01 +0100 |
| commit | 8f9bc93ba3299efbd8ad181481e6e2e87df7c066 (patch) | |
| tree | 93726d529d3a16ad94abe612672d954dd8d210b0 | |
| parent | 4f1cf73b4da42a8bc9147712e3c575356f7e4c30 (diff) | |
coverity#1242441 Identical code for different branches
before
commit 887b3a35628b349e7934f2ae7c220a1cc17c3e63
Date: Fri Jun 13 17:52:30 2014 +0200
loplugin:staticcall
this looked like
SwFtnFrm *pFtnFrm = bEndn ? pEndBoss->FindFtn( pSource, pFtn ) :
pFtnBoss->FindFtn( pSource, pFtn );
which superficially seemed to have different if else branches
Change-Id: I450c8080d503489902f16cc756ff9f7013971388
| -rw-r--r-- | sw/source/core/text/txtftn.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sw/source/core/text/txtftn.cxx b/sw/source/core/text/txtftn.cxx index e9ae8e28b4bd..068a41c9458d 100644 --- a/sw/source/core/text/txtftn.cxx +++ b/sw/source/core/text/txtftn.cxx @@ -461,8 +461,7 @@ void SwTxtFrm::RemoveFtn( const sal_Int32 nStart, const sal_Int32 nLen ) // -> Ftn wandert in den PrevFollow // beide muessen auf einer Seite/in einer Spalte stehen. - SwFtnFrm *pFtnFrm = bEndn ? SwFtnBossFrm::FindFtn( pSource, pFtn ) : - SwFtnBossFrm::FindFtn( pSource, pFtn ); + SwFtnFrm *pFtnFrm = SwFtnBossFrm::FindFtn(pSource, pFtn); if( pFtnFrm ) { |