Bug 18476 - cairo-directfb glyps not working properly
Summary: cairo-directfb glyps not working properly
Status: RESOLVED FIXED
Alias: None
Product: cairo
Classification: Unclassified
Component: directfb backend (show other bugs)
Version: 1.8.6
Hardware: Other Linux (All)
: highest critical
Assignee: Carl Worth
QA Contact: cairo-bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-11-10 22:13 UTC by swapnil
Modified: 2012-02-09 14:23 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
Screenshot with broken "Universität" text (6.58 KB, image/png)
2009-02-19 07:36 UTC, Erwin Rol
Details
Screenshot with broken "solarCity" text (7.31 KB, image/png)
2009-02-19 07:37 UTC, Erwin Rol
Details
Patch based on Cairo version 1.8.10 (309 bytes, patch)
2010-05-19 02:37 UTC, Gilles Bertal
Details | Splinter Review
Patch to fix font cache corruption. (787 bytes, patch)
2010-07-06 00:07 UTC, Peter Christoffersen
Details | Splinter Review

Description swapnil 2008-11-10 22:13:01 UTC
glyps are not working properly with directfb and showing broken
font for unicode (korean font)
Comment 1 swapnil 2008-11-11 02:02:45 UTC
(In reply to comment #0)
> glyps are not working properly with directfb and showing broken
> font for unicode (korean font)

#define DFB_SHOW_GLYPHS 0 is working properly,without glyps
in cairo-directfb-surafce.c
but #define DFB_SHOW_GLYPHS 1 is not working  properly i.e glyps
are not working when backend is directfb ,shows broken font (korean,japanese)
Comment 2 Erwin Rol 2009-02-19 07:35:16 UTC
I am having maybe the same problems. The two attached screenshots are from the same screen (just cropped to make the file smaller). The should say "Universität"  and "solarCity". As you can see the "t" in both is broken in exatly the same way. 

The problem goes away by setting #define DFB_SHOW_GLYPHS 0.

- Erwin
Comment 3 Erwin Rol 2009-02-19 07:36:05 UTC
Created attachment 23116 [details]
Screenshot with broken "Universität" text
Comment 4 Erwin Rol 2009-02-19 07:37:10 UTC
Created attachment 23117 [details]
Screenshot with broken "solarCity" text
Comment 5 Gilles Bertal 2010-05-19 02:37:19 UTC
Created attachment 35751 [details] [review]
Patch based on Cairo version 1.8.10

Attaching a trivial fix proposal based on Cairo version 1.8.10.
Comment 6 Peter Christoffersen 2010-07-06 00:06:20 UTC
(In reply to comment #5)
> Created an attachment (id=35751) [details]
> Patch based on Cairo version 1.8.10
> 
> Attaching a trivial fix proposal based on Cairo version 1.8.10.

The patch is not 100% correct. Cairo will keep resizing the font cache surface every time a new glyph is added to the font cache after it has filled the first line of glyphs.
I have attached a patch that fixes the prolem.
Comment 7 Peter Christoffersen 2010-07-06 00:07:36 UTC
Created attachment 36781 [details] [review]
Patch to fix font cache corruption.
Comment 8 Chris Wilson 2012-02-09 14:23:57 UTC
The wicked code is gone, long live the wicked code.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.