[Libreoffice-commits] core.git: svtools/source

[Khaled Hosny (via logerrit)]

 svtools/source/misc/sampletext.cxx |    6 ++++++
 1 file changed, 6 insertions(+)

New commits:
commit 18b39c8c9b35528c8fb172339126b613827d09aa
Author:     Khaled Hosny <kha...@aliftype.com>
AuthorDate: Mon Dec 26 00:25:54 2022 +0200
Commit:     خالد حسني <kha...@aliftype.com>
CommitDate: Sun Dec 25 23:15:30 2022 +0000

    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>

diff --git a/svtools/source/misc/sampletext.cxx 
b/svtools/source/misc/sampletext.cxx
index a181fdde238c..abcfde92b7a1 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)));