diff options
author | Khaled Hosny <khaledhosny@eglug.org> | 2016-12-08 07:04:27 +0200 |
---|---|---|
committer | Khaled Hosny <khaledhosny@eglug.org> | 2016-12-11 00:25:42 +0000 |
commit | d5e7e817980bfce29196dc55a20270201f77c673 (patch) | |
tree | 50975a27b4a25343d3a6fe151cd491fddf357da4 | |
parent | 869368e94dcfe393b0d06b1c1de18b5ce0d08d3d (diff) |
Drop a hack for Type 1, the future is here
The original issue for reference:
https://bz.apache.org/ooo/show_bug.cgi?id=36691
Change-Id: I3b68c2c9f7cb94445875ef5a005e73dcfe65fa3a
Reviewed-on: https://gerrit.libreoffice.org/31823
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Khaled Hosny <khaledhosny@eglug.org>
-rw-r--r-- | vcl/source/gdi/pdfwriter_impl.cxx | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/vcl/source/gdi/pdfwriter_impl.cxx b/vcl/source/gdi/pdfwriter_impl.cxx index 75ba6d1854b9..56f7d97ada26 100644 --- a/vcl/source/gdi/pdfwriter_impl.cxx +++ b/vcl/source/gdi/pdfwriter_impl.cxx @@ -8621,18 +8621,6 @@ void PDFWriterImpl::drawLayout( SalLayout& rLayout, const OUString& rText, bool } else aUnicodes.push_back(rText[pGlyphs[i]->mnCharPos]); - // #i36691# hack that is needed because currently the pGlyphs[] - // argument is ignored for embeddable fonts and so the layout - // engine's glyph work is ignored (i.e. char mirroring) - // TODO: a real solution would be to map the layout engine's - // glyphid (i.e. FreeType's synthetic glyphid for a Type1 font) - // back to unicode and then to embeddable font's encoding - if( (getReferenceDevice()->GetLayoutMode() & ComplexTextLayoutFlags::BiDiRtl) != ComplexTextLayoutFlags::Default ) - { - size_t nI = aUnicodes.size()-1; - for( int n = 0; n < nChars; n++, nI-- ) - aUnicodes[nI] = static_cast<sal_Ucs>(GetMirroredChar(aUnicodes[nI])); - } } else aUnicodes.push_back( 0 ); |