commit 18b39c8c9b35528c8fb172339126b613827d09aa
author Khaled Hosny <khaled@aliftype.com> Mon Dec 26 00:25:54 2022 +0200
committer خالد حسني <khaled@aliftype.com> Sun Dec 25 23:15:30 2022 +0000
tree e9bc4ad46797a05b57a034c2b453ddbd04fbf47a
parent e431417aa67d4b31f57e9d751b63e8e8cf002567

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: خالد حسني <khaled@aliftype.com>

svtools/source/misc/sampletext.cxx

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