diff options
author | Jan-Marek Glogowski <glogow@fbihome.de> | 2018-08-23 16:37:12 +0200 |
---|---|---|
committer | Jan-Marek Glogowski <glogow@fbihome.de> | 2018-08-23 19:10:56 +0200 |
commit | b0a5a564c359adc55f953c7545f52740e90a6e2e (patch) | |
tree | c38739d187eae4d7f77a4ae2bba73c9136bc04ec /vcl/unx/generic/print/glyphset.hxx | |
parent | b64f4847cceef666ba3c86950df729f3c94a1bd9 (diff) |
tdf#119454 don't advance PS glyphs
The glyphs from SalLayout already have the correct position, so
one doesn't have to advance them.
This is why PS glyphs were positioned "off-by-one", which is
especially visible, if your test document has words with spaces.
"dm d" was almost rendered / printed as "d md".
This is a regression from commit 2325f9ac789c ("fix bug in
GlyphSet::DrawGlyph"), which assumed the dead / unused code was
a bug, introduced by commit b157b82a6d92 ("Use GlyphItem in more
places"), which out of luck, seem to have been correct.
Since nobody uses the advance anywhere, just drop the parameter.
Change-Id: I40e8f99a98f84d48446a9190566af8cfe441cd88
Reviewed-on: https://gerrit.libreoffice.org/59510
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
Tested-by: Jan-Marek Glogowski <glogow@fbihome.de>
Diffstat (limited to 'vcl/unx/generic/print/glyphset.hxx')
-rw-r--r-- | vcl/unx/generic/print/glyphset.hxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/vcl/unx/generic/print/glyphset.hxx b/vcl/unx/generic/print/glyphset.hxx index ff9310523ce7..7c058dd3fc16 100644 --- a/vcl/unx/generic/print/glyphset.hxx +++ b/vcl/unx/generic/print/glyphset.hxx @@ -74,8 +74,7 @@ public: void DrawGlyph (PrinterGfx& rGfx, const Point& rPoint, - const sal_GlyphId nGlyphId, - const sal_Int32 nDelta); + const sal_GlyphId nGlyphId); void PSUploadFont (osl::File& rOutFile, PrinterGfx &rGfx, bool bAsType42, std::vector< OString >& rSuppliedFonts ); }; |