diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-10-28 23:51:21 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-10-29 05:59:15 +0100 |
| commit | 951a65923c088a1e19a4073f3c26a3b564a0a922 (patch) | |
| tree | 5e7191cc0783df304e5f7f28616dd31ee950a405 /vcl/win | |
| parent | cb7500ddb8de6c41fca84a0009ffe22240bb1845 (diff) | |
tdf#120703 PVS: V547 Expression is always true/false
Change-Id: I0516dc68cf7d451eafc044be8e50a66d2bddf15f
Reviewed-on: https://gerrit.libreoffice.org/62484
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'vcl/win')
| -rw-r--r-- | vcl/win/gdi/winlayout.cxx | 19 |
1 files changed, 4 insertions, 15 deletions
diff --git a/vcl/win/gdi/winlayout.cxx b/vcl/win/gdi/winlayout.cxx index e2b53261c9ef..71204b072c85 100644 --- a/vcl/win/gdi/winlayout.cxx +++ b/vcl/win/gdi/winlayout.cxx @@ -135,21 +135,10 @@ bool WinFontInstance::CacheGlyphToAtlas(HDC hDC, HFONT hFont, int nGlyphIndex, S UINT nPos = 0; - // FIXME: really I don't get why 'vertical' makes any difference [!] what does it mean !? - if (aElement.mbVertical) - { - aElement.maLocation.SetLeft(0); - aElement.maLocation.SetRight(nBitmapWidth); - aElement.maLocation.SetTop(nPos); - aElement.maLocation.SetBottom( nPos + aGlyphAdv[0] + aElement.maLeftOverhangs ); - } - else - { - aElement.maLocation.SetLeft(nPos); - aElement.maLocation.SetRight(aEnds[0]); - aElement.maLocation.SetTop(0); - aElement.maLocation.SetBottom( bounds.getHeight() + aElement.getExtraSpace() ); - } + aElement.maLocation.SetLeft(nPos); + aElement.maLocation.SetRight(aEnds[0]); + aElement.maLocation.SetTop(0); + aElement.maLocation.SetBottom(bounds.getHeight() + aElement.getExtraSpace()); nPos = aEnds[0]; OpenGLCompatibleDC aDC(rGraphics, 0, 0, nBitmapWidth, nBitmapHeight); |