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 380094 - Fix split matra rendering bug for Kannada language
Fix split matra rendering bug for Kannada language
Status: RESOLVED FIXED
Product: pango
Classification: Platform
Component: indic
unspecified
Other All
: Normal normal
: ---
Assigned To: Pango Indic
pango-maint
Depends on:
Blocks:
 
 
Reported: 2006-11-28 10:48 UTC by H P Nadig
Modified: 2008-06-23 21:30 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16


Attachments
The patch solves the problem , mentioned in the bug (731 bytes, patch)
2007-01-08 06:58 UTC, Ramakrishna
needs-work Details | Review

Description H P Nadig 2006-11-28 10:48:39 UTC
Please describe the problem:
Pango rendering splits the glyphs U+0C8E (ಎ) and U+0C82 (anusvaara)  when they appear in sequence. 



Steps to reproduce:
1. Type the word ಎಂಬ ("eMba" with SCIM-m17n kn-itrans selection).  
2. Paste the same on Kate or Kwrite. 
 



Actual results:
You'll notice the difference in rendering of the split maatra.

Expected results:
Pango should render the Anusvaara (U+0C82) beside U+0C8E (ಎ))  without dotted circles. 

Does this happen every time?
Yes. 

Other information:
http://sampada.net/pango-kannada-rendering-bug.zip
contains two images - one taken on QT based application that shows correct rendering and the other showing the bad rendering on Pango (the files are appropriately named). Take a note of just the first two letters on those.
Comment 1 Shashishekhar S 2006-11-29 14:20:52 UTC
I'm confirming the bug mentioned as above.  I copied the above word "ಎಂಬ" into gedit, gaim and the rendering remained the same.
Comment 2 skanchi 2006-12-09 10:41:28 UTC
(In reply to comment #1)
> I'm confirming the bug mentioned as above.  I copied the above word "ಎಂಬ"
> into gedit, gaim and the rendering remained the same.
> 

i confirm the same. 
Comment 3 Ramakrishna 2007-01-08 06:58:01 UTC
Created attachment 79710 [details] [review]
The patch solves the problem , mentioned in the bug

patch taken against pango-1.14.9
Comment 4 Ramakrishna 2007-01-08 07:04:46 UTC
Was previously defined as an unknown character (_xx) in the indic-ot-class-table.c file.
Comment 5 omshivaprakash 2007-05-05 22:30:00 UTC
Did any one get a chance to apply this patch?
Comment 6 omshivaprakash 2007-08-31 22:18:38 UTC
The patch released on 2007-01-08 by Mr. Ramakrishna works pretty well with the new development release too Pango Version 1.18.1.
Comment 7 Praveen A 2008-03-18 15:52:24 UTC
Same bug is also reported at
http://bugzilla.gnome.org/show_bug.cgi?id=481205

This is an important bug with a long pending trivial patch verified by so many reporters and discussed in so many places. Please commit it and close it.
Comment 8 shankar 2008-05-22 05:59:48 UTC
The patch has been submitted about 8 months back and people have certified that its working fine, but still, there is hardly any answer by the upstream!!? Can we know what is the roadblock for this patch to get comitted/accepted!!?
Some thing more should be done from our side!!? 
Comment 9 Rahul Bhalerao 2008-06-05 12:15:58 UTC
Behdad, this patch is very simple and well tested as well now. I think it is worth committing now. 
Comment 10 Behdad Esfahbod 2008-06-23 21:24:44 UTC
The patch is simple, well tested, but no one even tried explaining *why* it's the correct patch.  In fact, in this simple case my untrained Indic eyes see that it's not the correct patch.  Attaching correct one.
Comment 11 Behdad Esfahbod 2008-06-23 21:29:04 UTC
Correct patch.  The thing is, U+0C8D is Reserved, but the Indic table was marked as if U+0C8E is such.  Swapping the classes.


Index: indic-ot-class-tables.c
===================================================================
--- indic-ot-class-tables.c	(revision 2657)
+++ indic-ot-class-tables.c	(working copy)
@@ -196,7 +196,7 @@ static const IndicOTCharClass teluCharCl
  */
 static const IndicOTCharClass kndaCharClasses[] =
 {
-    _xx, _xx, _mp, _mp, _xx, _iv, _iv, _iv, _iv, _iv, _iv, _iv, _iv, _iv, _xx, _iv, /* 0C80 - 0C8F */
+    _xx, _xx, _mp, _mp, _xx, _iv, _iv, _iv, _iv, _iv, _iv, _iv, _iv, _xx, _iv, _iv, /* 0C80 - 0C8F */
     _iv, _xx, _iv, _iv, _iv, _bb, _bb, _bb, _bb, _bb, _bb, _bb, _bb, _bb, _bb, _bb, /* 0C90 - 0C9F */
     _bb, _bb, _bb, _bb, _bb, _bb, _bb, _bb, _bb, _xx, _bb, _bb, _bb, _bb, _bb, _bb, /* 0CA0 - 0CAF */
     _rb, _ct, _bb, _bb, _xx, _bb, _bb, _bb, _bb, _bb, _xx, _xx, _xx, _xx, _dr, _da, /* 0CB0 - 0CBF */
Comment 12 Behdad Esfahbod 2008-06-23 21:30:15 UTC
2008-06-23  Behdad Esfahbod  <behdad@gnome.org>

        Bug 380094 – Fix split matra rendering bug for Kannada language

        * modules/indic/indic-ot-class-tables.c: Fix Indic class for
        U+0C8D and U+0C8E.