diff options
author | Mike Kaganski <mike.kaganski@collabora.com> | 2017-12-20 05:29:58 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-12-20 07:13:01 +0100 |
commit | 9c50b0ed877c4179b5820a32be742b2190d927ee (patch) | |
tree | d27af29d2553ac6cb7988065b2dc7f61d8c9d313 /formula | |
parent | 495bb32c23be5acec3783240eb03ccfc5aca44cd (diff) |
loplugin:unusedindex
Change-Id: Id69dcfde5c8b5d39c272916e7c1bcc649f720888
Reviewed-on: https://gerrit.libreoffice.org/46829
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'formula')
-rw-r--r-- | formula/source/core/api/FormulaCompiler.cxx | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/formula/source/core/api/FormulaCompiler.cxx b/formula/source/core/api/FormulaCompiler.cxx index da9ac30cc136..b0dca4d0864f 100644 --- a/formula/source/core/api/FormulaCompiler.cxx +++ b/formula/source/core/api/FormulaCompiler.cxx @@ -817,18 +817,16 @@ FormulaCompiler::OpCodeMapPtr FormulaCompiler::CreateOpCodeMap( FormulaGrammar::GRAM_EXTERNAL, bEnglish), FormulaGrammar::CONV_UNSPECIFIED))); SvtSysLocale aSysLocale; const CharClass* pCharClass = (xMap->isEnglish() ? nullptr : aSysLocale.GetCharClassPtr()); - FormulaOpCodeMapEntry const * pArr2 = rMapping.getConstArray(); - FormulaOpCodeMapEntry const * const pStop = pArr2 + rMapping.getLength(); - for ( ; pArr2 < pStop; ++pArr2) + for (auto const& rMapEntry : rMapping) { - OpCode eOp = OpCode(pArr2->Token.OpCode); + OpCode eOp = OpCode(rMapEntry.Token.OpCode); if (eOp != ocExternal) - xMap->putOpCode( pArr2->Name, eOp, pCharClass); + xMap->putOpCode( rMapEntry.Name, eOp, pCharClass); else { OUString aExternalName; - if (pArr2->Token.Data >>= aExternalName) - xMap->putExternal( pArr2->Name, aExternalName); + if (rMapEntry.Token.Data >>= aExternalName) + xMap->putExternal( rMapEntry.Name, aExternalName); else { SAL_WARN( "formula.core", "FormulaCompiler::CreateOpCodeMap: no Token.Data external name"); |