diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-08-11 13:53:37 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-08-13 08:34:28 +0200 |
commit | eb841c06f879df11142c659b372e365c361a231b (patch) | |
tree | b1e67bd2895569325d11392a3d67769e3061068c /lotuswordpro/source/filter/benval.cxx | |
parent | ff4d6a7e5b166bb98bbe8d0f59cc4db36e7002aa (diff) |
loplugin:returnconstant in linguistic,lotuswordpro
Change-Id: If5c7d936020e5470b09c2fe54d41fb006c296a34
Reviewed-on: https://gerrit.libreoffice.org/58877
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'lotuswordpro/source/filter/benval.cxx')
-rw-r--r-- | lotuswordpro/source/filter/benval.cxx | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/lotuswordpro/source/filter/benval.cxx b/lotuswordpro/source/filter/benval.cxx index 9cd7404f996d..2d93626ececd 100644 --- a/lotuswordpro/source/filter/benval.cxx +++ b/lotuswordpro/source/filter/benval.cxx @@ -78,7 +78,6 @@ void CBenValue::ReadValueData(void * pReadBuffer, size_t Offset, size_t Amt, size_t* pAmtRead) { - BenError Err; size_t SegOffset = 0; *pAmtRead = 0; CBenValueSegment * pCurrSeg = nullptr; @@ -114,13 +113,9 @@ CBenValue::ReadValueData(void * pReadBuffer, size_t Offset, } else { - if ((Err = pContainer->SeekToPosition(pCurrSeg->GetPosition() + - OffsetIntoSeg)) != BenErr_OK) - return; - - if ((Err = pContainer->Read(pBuffer, AmtThisSeg, - &AmtReadThisSeg)) != BenErr_OK) - return; + pContainer->SeekToPosition(pCurrSeg->GetPosition() + + OffsetIntoSeg); + pContainer->Read(pBuffer, AmtThisSeg, &AmtReadThisSeg); } *pAmtRead += AmtReadThisSeg; |