diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-02-11 15:56:21 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-02-12 10:31:25 +0100 |
| commit | f147b160aef1735d34c488353a7d4b875788a7e1 (patch) | |
| tree | ab3bd613cf016cde906b3fdcbaab4246ca699e1e /sdext/source/pdfimport/pdfparse/pdfentries.cxx | |
| parent | tdf#130440: Add UItest (diff) | |
| download | core-f147b160aef1735d34c488353a7d4b875788a7e1.tar.gz core-f147b160aef1735d34c488353a7d4b875788a7e1.zip | |
clang-analyzer-deadcode.DeadStores
Change-Id: Ifa384933569b27d0d08eb479bb95b799163ae386
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88450
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sdext/source/pdfimport/pdfparse/pdfentries.cxx')
| -rw-r--r-- | sdext/source/pdfimport/pdfparse/pdfentries.cxx | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/sdext/source/pdfimport/pdfparse/pdfentries.cxx b/sdext/source/pdfimport/pdfparse/pdfentries.cxx index 88cb3e1789ab..f8e24addeacb 100644 --- a/sdext/source/pdfimport/pdfparse/pdfentries.cxx +++ b/sdext/source/pdfimport/pdfparse/pdfentries.cxx @@ -538,13 +538,11 @@ void PDFDict::insertValue( const OString& rName, std::unique_ptr<PDFEntry> pValu else { unsigned int nSub = m_aSubElements.size(); - bool bFound = false; - for( unsigned int i = 0; i < nSub && !bFound; i++ ) + for( unsigned int i = 0; i < nSub; i++ ) if( m_aSubElements[i].get() == it->second ) { m_aSubElements[i] = std::move(pValue); pValueTmp = m_aSubElements[i].get(); - bFound = true; break; } } |