svtools/source/misc/sampletext.cxx | 6 ++++++
1 file changed, 6 insertions(+)
New commits:
commit ac3bf1039f0a4dd767bc751e7c4cfa08fba6567d
Author: Khaled Hosny <kha...@aliftype.com>
AuthorDate: Mon Dec 26 00:25:54 2022 +0200
Commit: Andras Timar <andras.ti...@collabora.com>
CommitDate: Tue Jan 10 11:25:43 2023 +0100
tdf#88484: Show sample text for Abyssinica SIL font
The font sets Arabic Presentation Forms-B bit, probably because it
includes U+FEFF (Zero Width Space) which happens to be in that block but
it isn’t an Arabic code point. Filter this bit out if we already decided
it isn’t an Arabic font.
Change-Id: I9935c793b8a7592eb92db59bc75de65b81ab99fe
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144819
Tested-by: Jenkins
Reviewed-by: خالد حسني <kha...@aliftype.com>
(cherry picked from commit 18b39c8c9b35528c8fb172339126b613827d09aa)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144797
Reviewed-by: Adolfo Jayme Barrientos <fit...@ubuntu.com>
diff --git a/svtools/source/misc/sampletext.cxx
b/svtools/source/misc/sampletext.cxx
index c59f5a7a43ec..52351e51dd3a 100644
--- a/svtools/source/misc/sampletext.cxx
+++ b/svtools/source/misc/sampletext.cxx
@@ -1122,6 +1122,12 @@ namespace
aMasked.set(vcl::UnicodeCoverage::GREEK_EXTENDED, false);
aMasked.set(vcl::UnicodeCoverage::GREEK_AND_COPTIC, false);
+ // tdf#88484
+ // Some fonts set the Arabic Presentation Forms-B bit because they
+ // support U+FEFF (Zero Width Space) which happens to be in that block
+ // but it isn’t an Arabic code point. By the time we reach here we
+ // decided this isn’t an Arabic font, so it should be safe.
+ aMasked.set(vcl::UnicodeCoverage::ARABIC_PRESENTATION_FORMS_B, false);
if (aMasked.count() == 1)
return
otCoverageToScript(static_cast<UnicodeCoverageEnum>(find_first(aMasked)));
