vcl/source/gdi/CommonSalLayout.cxx | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
New commits:
commit d183e4b59207378eeef4ac602a42dd12b16ffec4
Author: Jan-Marek Glogowski <glo...@fbihome.de>
AuthorDate: Sat Apr 27 21:53:21 2019 +0200
Commit: Jan-Marek Glogowski <glo...@fbihome.de>
CommitDate: Sun Apr 28 20:37:05 2019 +0200
Just apply Asian kerning to unmodified glyphs
IMHO the old code was doing the opposite of the comment. But the
comment makes more sense then the code, so adjust the code
accordingly.
OTOH just this code was added in commit f89d70de85a6 ("INTEGRATION:
CWS vcl27 (1.30.48); FILE MERGED 2004/09/09 12:33:53 hdu
1.30.48.1: #116489# only apply asian kerning on default-formatted
text"). Maybe still a bug? Can't check the bug tracker, as this is
pre-Apache, and so can't verify the old bug :-(
Change-Id: I550d1524d3e82f7e853820d3e5ddab8cc62228f9
Reviewed-on: https://gerrit.libreoffice.org/71441
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski <glo...@fbihome.de>
diff --git a/vcl/source/gdi/CommonSalLayout.cxx
b/vcl/source/gdi/CommonSalLayout.cxx
index 872603228508..5ef6992cabdc 100644
--- a/vcl/source/gdi/CommonSalLayout.cxx
+++ b/vcl/source/gdi/CommonSalLayout.cxx
@@ -208,12 +208,10 @@ void GenericSalLayout::AdjustLayout(ImplLayoutArgs& rArgs)
ApplyDXArray(rArgs);
else if (rArgs.mnLayoutWidth)
Justify(rArgs.mnLayoutWidth);
-
// apply asian kerning if the glyphs are not already formatted
- if ((rArgs.mnFlags & SalLayoutFlags::KerningAsian)
- && !(rArgs.mnFlags & SalLayoutFlags::Vertical))
- if ((rArgs.mpDXArray != nullptr) || (rArgs.mnLayoutWidth != 0))
- ApplyAsianKerning(rArgs.mrStr);
+ else if ((rArgs.mnFlags & SalLayoutFlags::KerningAsian)
+ && !(rArgs.mnFlags & SalLayoutFlags::Vertical))
+ ApplyAsianKerning(rArgs.mrStr);
}
void GenericSalLayout::DrawText(SalGraphics& rSalGraphics) const
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
