diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-03-08 13:48:09 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-03-08 18:23:53 +0100 |
| commit | c774ad610eb828742a3eea050e5a7571ba3b00be (patch) | |
| tree | 5563215fe7b481f99efc96017d61ecd5c7a59072 /unotools | |
| parent | 24d0704fa468b300558a3f904ae853fcb7fca312 (diff) | |
loplugin:constantparam in svtools..unotools
Change-Id: I6e72fbe44dcb65ee5162448e9a72e6437d56b044
Reviewed-on: https://gerrit.libreoffice.org/50948
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'unotools')
| -rw-r--r-- | unotools/source/misc/fontcvt.cxx | 8 |
1 files changed, 3 insertions, 5 deletions
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" ) |