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 326960 - hex box drawing for win32 and atsui backends of cairo
hex box drawing for win32 and atsui backends of cairo
Status: RESOLVED FIXED
Product: pango
Classification: Platform
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: pango-maint
pango-maint
Depends on:
Blocks:
 
 
Reported: 2006-01-14 13:32 UTC by Behdad Esfahbod
Modified: 2006-06-07 18:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
show unknown glyph of indic (817 bytes, patch)
2006-01-23 07:01 UTC, LingNing Zhang
committed Details | Review

Description Behdad Esfahbod 2006-01-14 13:32:03 UTC
The pangocairo-win32 and pangocairo-atsui backends need to support hex box drawing.  That's pretty straightforward given the work that went in during bug #313551.
Comment 1 LingNing Zhang 2006-01-23 06:14:56 UTC
Pango can not show unknown characters of indic glyph. I find that there is not the code about supporting unknown glyph in indic module.
Comment 2 LingNing Zhang 2006-01-23 07:01:00 UTC
Created attachment 57900 [details] [review]
show unknown glyph of indic

I wrote a patch about supporting to show unknown glyph of indic.
I only modified ./modules/indic/indic-fc.c
Comment 3 Behdad Esfahbod 2006-01-23 12:44:53 UTC
Thanks, committed after some change.  Please do not use current bugreports for issues not directly related to them.  This issue doesn't have anything to do with win32 or atsui...

2006-01-23  Behdad Esfahbod  <behdad@gnome.org>

        * modules/basic/basic-fc.c: Set cluster only on base chars, like the
        Arabic module does.

        * modules/indic/indic-fc.c: Draw an unknown glyph on missing glyphs,
        like other modules do. (part of bug #326960, LingNing Zhang)

Comment 4 Behdad Esfahbod 2006-06-07 06:09:25 UTC
Richard, please test and feel free to commit to pango-1-12 branch.

2006-06-07  Behdad Esfahbod  <behdad@gnome.org>

        Bug 326960 – hex box drawing for win32 and atsui backends of cairo

        Hans already fixed win32.

        * pango/pangocairo-atsuifont.c (compute_glyph_extents): Handle
        PANGO_GLYPH_UNKNOWN_FLAG.

Comment 5 Richard Hult 2006-06-07 17:57:15 UTC
Thanks! I've tested this and it builds and runs at least. I'm not sure what the expected output is, I tried putting some random unicode characters in an entry and got boxes (but no hex in them).
Comment 6 Behdad Esfahbod 2006-06-07 18:09:02 UTC
Ok, that's probably good enough for now. The basic-atsui shaper should be changed to use hexbox glyph ids for unknown glyphs...