+++ This bug was initially created as a clone of Bug #223169 +++ +++ This bug was initially created as a clone of Bug #220874 +++ Description of problem: 3 more combinations has problem with pango/icu (different for icu) 3) [അപ്ഗ്റേഡ്] 0D05 + 0D2A + 0D4D + 0D17 + 0D4D + 0D31 + 0d47 + 0D21 + 0D4D 4) [വ്യക്തം] 0D35 + 0D4D + 0D2F + 0D15 + 0D4D + 0D24 + 0d02 5) [ഇന്സ്റ്റോള്] 0D07 + 0D28 + 0D4D + 200D + 0D38 + 0D4D + 0D31 + 0D4D + 0D31 + 0D4B + 0D33 + 0D4D + 200D Version-Release number of selected component (if applicable): pango-1.15.0 How reproducible: Everytime Steps to Reproduce: 1. open gedit/oowriter 2. type give combination (use att text file) 3. open those in kedit Actual results: as in gedit/oowriter Expected results: as in kedit Additional info: >attachment 1) Screenshot attached for kedit, gedit, oowriter 2) simple text file for those 3 combination >Ref this bug is part of bug #220866 -- Additional comment from aalam on 2006-12-28 02:19 EST -- Created an attachment (id=144443) Test Case (text file) -- Additional comment from aalam on 2006-12-28 02:23 EST -- Created an attachment (id=144444) Image at RHEL5Client machine -- Additional comment from lizhang on 2007-01-17 23:55 EST -- Created an attachment (id=145879) my patch -- Additional comment from lizhang on 2007-02-07 20:56 EST -- This bug and bug223170 have 4 problems, this patch only can fix one problem, so it need be modified. -- Additional comment from lizhang on 2007-06-13 02:40 EST -- I wrote a patch for fixing bug238896, bug223169 and bug223170. And I modified indic_shape_engine( ) in this patch, let it handle one by one syllable, not handle all syllables together. It can fix the three bugs only by this way. The patch is below: https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=156846 -- Additional comment from lizhang on 2007-09-27 23:10 EST -- *** Bug 288911 has been marked as a duplicate of this bug. *** -- Additional comment from apeter on 2008-01-25 06:46 EST -- The problem with all three combinations still exists. Please have a look in the bug. Pango version: pango-1.19.3-1.fc9 Thanks Ani -- Additional comment from apeter on 2008-02-04 01:31 EST -- Problem still exists and necessary action is highly appreciated. Slight change in the first combination mentioned in the problem Correct combination is: [അപ്ഗ്രേഡ്] 0D05 + 0D2A + 0D4D + 0D17 + 0D4D + 0D30 + 0d47 + 0D21 + 0D4D. Thanks Ani
Hello again Caolan, All is fixed in icu-3.8.1-4.fc9 except for combinations like [അപ്ഗ്രേഡ്] 0D05 + 0D2A + 0D4D + 0D17 + 0D4D + 0D30 + 0d47 + 0D21 + 0D4D ie, If a consonant with 0D4D is followed by the combination of (a consonant + 0D4D + 0D30), the glyph of combination of (a consonant + 0D4D + 0D30) appears wrong. please find the screenshot attached. Thanks Ani
Created attachment 293869 [details] Scrreshot for the problematic combination
So this is test-case number 252 of http://batman.brisbane.redhat.com/~indic/IndicTC/lang/ml_IN/font/IndicFontTestCaseGSUB-Malayalam.html ? If so can we change the text of that test to be ... A PA VIRAMA GA VIRAMA RA VOWEL SIGN EE DDA VIRAMA and not A PA VIRAMA GA VIRAMA RRA VOWEL SIGN EE DDA VIRAMA to match the unicode code points used. I think I can hack in a special case for this sort of thing in >= icu-3.8.1-5, but I really should make some time to figure out all this from scratch and re-examine all the malayalam specific stuff and see why it has become so much more complex than all the other indics, perhaps in conjunction with testing icu with the microsoft malayalam font instead of/as well as the lohit one.