From c774ad610eb828742a3eea050e5a7571ba3b00be Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Thu, 8 Mar 2018 13:48:09 +0200 Subject: loplugin:constantparam in svtools..unotools Change-Id: I6e72fbe44dcb65ee5162448e9a72e6437d56b044 Reviewed-on: https://gerrit.libreoffice.org/50948 Tested-by: Jenkins Reviewed-by: Noel Grandin --- unotools/source/misc/fontcvt.cxx | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'unotools') diff --git a/unotools/source/misc/fontcvt.cxx b/unotools/source/misc/fontcvt.cxx index dfaafd522a09..616e1b23d813 100644 --- a/unotools/source/misc/fontcvt.cxx +++ b/unotools/source/misc/fontcvt.cxx @@ -1306,7 +1306,7 @@ struct RecodeTable { const char* pOrgName; ConvertChar aCvt;}; static const RecodeTable aStarSymbolRecodeTable[] = { // the first two entries must be StarMath and StarBats; do not reorder! - // reason: fgrep for FONTTOSUBSFONT_ONLYOLDSOSYMBOLFONTS + // reason: see CreateFontToSubsFontConverter method {"starbats", {aStarBatsTab, "StarSymbol", nullptr}}, {"starmath", {aStarMathTab, "StarSymbol", nullptr}}, @@ -1388,9 +1388,7 @@ FontToSubsFontConverter CreateFontToSubsFontConverter( const OUString& rOrgName, if ( nFlags & FontToSubsFontFlags::IMPORT ) { - int nEntries = SAL_N_ELEMENTS(aStarSymbolRecodeTable); - if ( nFlags & FontToSubsFontFlags::ONLYOLDSOSYMBOLFONTS ) // only StarMath+StarBats - nEntries = 2; + int nEntries = 2; // only StarMath+StarBats for( int i = 0; i < nEntries; ++i ) { const RecodeTable& r = aStarSymbolRecodeTable[i]; @@ -1403,7 +1401,7 @@ FontToSubsFontConverter CreateFontToSubsFontConverter( const OUString& rOrgName, } else { - // TODO: FontToSubsFontFlags::ONLYOLDSOSYMBOLFONTS + // TODO: only StarMath+StarBats if( aName == "starsymbol" ) pCvt = &aImplStarSymbolCvt; else if( aName == "opensymbol" ) -- cgit