dev/null |binary
sw/qa/extras/ww8export/ww8export.cxx | 6 ------
sw/source/filter/ww8/ww8par6.cxx | 5 -----
3 files changed, 11 deletions(-)
New commits:
commit d8c386593e42e1f0cce52d052b1009c59e75afa2
Author: Khaled Hosny <khaledho...@eglug.org>
Date: Thu Nov 17 15:03:39 2016 +0200
tdf#103944: Fix symbol font remapping
This reverts:
commit 8556cd881270823865662e9a7700da58d11c2785
Author: Miklos Vajna <vmik...@collabora.co.uk>
Date: Thu Mar 6 09:48:54 2014 +0100
cp#1000039 DOC import: ignore symbol charset of the symbol font
Otherwise characters unhandled by our OpenSymbol font (like Arabic 0-9
numbers) won't be rendered using an other font, as no other font
provides the symbol charset. Do this in
SwWW8ImplReader::GetFontParams(), where we already have font name ->
font family mappings for a few well-known fonts.
The DOCX filter does the same for quite some time, and that's how Arabic
numbers in text using the Symbol font were rendered, instead of little
rectangles.
The reverted commit prevented remapping symbols supported by OpenSymbol,
and it seems to have worked incidentally because of the fallback to the
âStandard Symbols Lâ Type 1 font which we longer support. The bug doc is
broken in master with or without this commit, but reverting this fixes
tdf#103944.
Change-Id: I17ac699fc5987e11e5c9e490895fc3c4967d3127
Reviewed-on: https://gerrit.libreoffice.org/30932
Tested-by: Jenkins <c...@libreoffice.org>
Reviewed-by: Miklos Vajna <vmik...@collabora.co.uk>
diff --git a/sw/qa/extras/ww8export/data/cp1000039.doc
b/sw/qa/extras/ww8export/data/cp1000039.doc
deleted file mode 100644
index 70cad3c..0000000
Binary files a/sw/qa/extras/ww8export/data/cp1000039.doc and /dev/null differ
diff --git a/sw/qa/extras/ww8export/ww8export.cxx
b/sw/qa/extras/ww8export/ww8export.cxx
index d3d8733..a897984 100644
--- a/sw/qa/extras/ww8export/ww8export.cxx
+++ b/sw/qa/extras/ww8export/ww8export.cxx
@@ -347,12 +347,6 @@ DECLARE_WW8EXPORT_TEST(testCp1000044, "cp1000044.doc")
CPPUNIT_ASSERT_EQUAL(false, bool(xStorable->isReadonly()));
}
-DECLARE_WW8EXPORT_TEST(testCp1000039, "cp1000039.doc")
-{
- // This was RTL_TEXTENCODING_SYMBOL, causing "1" rendered as a placeholder
rectangle.
- CPPUNIT_ASSERT_EQUAL(sal_Int16(RTL_TEXTENCODING_DONTKNOW),
getProperty<sal_Int16>(getRun(getParagraph(1), 1), "CharFontCharSet"));
-}
-
DECLARE_WW8EXPORT_TEST(testBorderColours, "bordercolours.doc")
{
// The following 6 colours can only be represented with WW9 (Word 2000)
diff --git a/sw/source/filter/ww8/ww8par6.cxx b/sw/source/filter/ww8/ww8par6.cxx
index 11850a0..f28b3a8 100644
--- a/sw/source/filter/ww8/ww8par6.cxx
+++ b/sw/source/filter/ww8/ww8par6.cxx
@@ -3467,11 +3467,6 @@ bool SwWW8ImplReader::GetFontParams( sal_uInt16 nFCode,
FontFamily& reFamily,
reCharSet = rtl_getTextEncodingFromWindowsCharset( pF->chs );
}
- // Make sure charset is not set in case of Symbol, otherwise in case a
- // character is missing in our OpenSymbol, the character won't be replaced.
- if (reCharSet == RTL_TEXTENCODING_SYMBOL && rName == "Symbol")
- reCharSet = RTL_TEXTENCODING_DONTKNOW;
-
// make sure Font Family Code is set correctly
// at least for the most important fonts
// ( might be set wrong when Doc was not created by
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
