After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 536017 - Remove SF_MPRE_FIXUP from Sinhala script flags
Remove SF_MPRE_FIXUP from Sinhala script flags
Status: RESOLVED FIXED
Product: pango
Classification: Platform
Component: indic
1.21.x
Other Linux
: Normal normal
: ---
Assigned To: Pango Indic
pango-maint
Depends on:
Blocks:
 
 
Reported: 2008-06-01 05:35 UTC by Harshula Jayasuriya
Modified: 2008-12-06 12:57 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
Patch to remove SF_MPRE_FIXUP from Sinhala script flags. (621 bytes, patch)
2008-06-01 05:38 UTC, Harshula Jayasuriya
committed Details | Review

Description Harshula Jayasuriya 2008-06-01 05:35:54 UTC
Hi,

Background
==========
Harsha Senanayake's Pango patch for adding Sinhala support was
submitted to Pango by Anuradha Ratnaweera in Bug 153517. The
aforementioned patch was mostly a simple copy-modify-paste of the
Malayalam/Tamil script support code, including worst case expansion
and script flags. Sinhala orthography is subtly different from both
the North Indian and South Indian orthographies.

1) North Indian Vs Sinhala
--------------------------
In North Indian orthographies a consonant cluster (Cons-Virama-Cons)
results in an implicit conjunct glyph therefore the ICU/Pango code denotes
this sequence of three codepoints as a 'matra'. This is not the case with
Sinhala where conjuncts must be *explicitly* created. In Sinhala a
sequence of Cons-Virama-Cons shall result in two 'matras', Cons-Virama
and Cons.

2) South Indian Vs Sinhala
--------------------------
Sinhala orthography has three ways of writing Cons-Virama-Cons - 2
separate glyphs, a conjunct, 2 touching letters. The last two ways
require the explicit use of ZWJ. My understanding is that South Indian
orthographies do not behave this way.


Issues resulting from (1) were fixed in Bug 161981 by introducing a
CC_AL_LAKUNA character class to Pango. However, CC_AL_LAKUNA was only
recently added to ICU (http://bugs.icu-project.org/trac/ticket/6232).
During the process, I realised SF_MPRE_FIXUP was inappropriate for
Sinhala because of (2).

In Pango, SF_MPRE_FIXUP in Sinhala script flags does not appear to have
a negative affect on layout, much like the incorrect worst case expansion
(Bug 385321) did not. However, technically it should never have been
included in the first place.

I'll attach a patch to remove SF_MPRE_FIXUP from the Sinhala script flag.

Regards,
Harshula
Comment 1 Harshula Jayasuriya 2008-06-01 05:38:37 UTC
Created attachment 111877 [details] [review]
Patch to remove SF_MPRE_FIXUP from Sinhala script flags.
Comment 2 Behdad Esfahbod 2008-06-18 21:31:32 UTC
2008-06-18  Behdad Esfahbod  <behdad@gnome.org>

        Bug 536017 – Remove SF_MPRE_FIXUP from Sinhala script flags
        Patch by Harshula

        * modules/indic/indic-ot-class-tables.c: Just freakin' do it.
        I have no idea if it's right or wrong.  Just that I've been
        pestered enough to do it.  Time will tell...

Comment 3 Danishka Navin 2008-12-06 12:57:44 UTC
I am using Fedora 10 (yes, with gnome desktop) right now and rendering is fine for me. :)