diff options
author | Caolán McNamara <caolanm@redhat.com> | 2021-04-14 09:57:33 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2021-04-14 12:39:01 +0200 |
commit | 0bbd74f93a6f37bf880981862437f892f43452d0 (patch) | |
tree | bc02d46c0cd3427975c9937d442c0b9569e4152a /vcl/unx/generic/fontmanager/fontconfig.cxx | |
parent | 4ed7a2c8af03bc0f45df1f03fd160ccbf045ed4f (diff) |
getFontOptions is always called with pitch of PITCH_DONTKNOW
getFontOptions is called from only one location and there
the pitch value is left at its default PITCH_DONTKNOW
which is suspicious but that's the way it currently is
Change-Id: I7e4a487b800a0111133a0edd472731ffa5122177
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/114076
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'vcl/unx/generic/fontmanager/fontconfig.cxx')
-rw-r--r-- | vcl/unx/generic/fontmanager/fontconfig.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/vcl/unx/generic/fontmanager/fontconfig.cxx b/vcl/unx/generic/fontmanager/fontconfig.cxx index 855868a022e4..17ca8f46ccae 100644 --- a/vcl/unx/generic/fontmanager/fontconfig.cxx +++ b/vcl/unx/generic/fontmanager/fontconfig.cxx @@ -1113,7 +1113,8 @@ std::unique_ptr<FontConfigFontOptions> PrintFontManager::getFontOptions(const Fa if( !sFamily.isEmpty() ) FcPatternAddString(pPattern, FC_FAMILY, reinterpret_cast<FcChar8 const *>(sFamily.getStr())); - addtopattern(pPattern, rInfo.m_eItalic, rInfo.m_eWeight, rInfo.m_eWidth, rInfo.m_ePitch); + // TODO: ePitch argument of always PITCH_DONTKNOW is suspicious + addtopattern(pPattern, rInfo.m_eItalic, rInfo.m_eWeight, rInfo.m_eWidth, PITCH_DONTKNOW); FcPatternAddDouble(pPattern, FC_PIXEL_SIZE, nSize); FcConfigSubstitute(pConfig, pPattern, FcMatchPattern); |