bugGNUstep - Bugs: bug #12129, advancementForGlyph in Win32...

Group
 
 

bug #12129: advancementForGlyph in Win32 Backend works incorrectly.

Submitter:  None
Submitted:  Thu 24 Feb 2005 02:07:16 PM UTC
   
 
Category:  Backend Severity:  3 - Normal
Item Group:  Bug Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 30 Jan 2009 07:20:30 PM UTC, comment #4: 

closing this as there has been no feedback in two years.

Richard Frith-Macdonald <CaS>
Group Member
Thu 01 Feb 2007 07:13:18 AM UTC, comment #3: 

I added calls to GetGlyphIndices() in a couple of places ... should serve to get advancement and bounding rect information as long as an NSGlyph is a unichar and a microsoft glyph corresponding to the unichar exists.
I'm provisionally marking this as 'fixed' since it looks as good a fix as we can do without someone revisiting the whole text system in the gui as well as the backends.

Richard Frith-Macdonald <CaS>
Group Member
Mon 07 Mar 2005 10:07:53 AM UTC, comment #2: 

I think even the recommended fix from the last comment would not be sufficent. As far as I can see the code here mixes glyph indexes and characters. Fixing just one more case wont help, as [drawGlyphs:length:onDC:at:] would than be broken.
What we could do as a short time solution is add a call to  GetGlyphIndices() or reenable the original GetCharWidthFloat() call.

Fred Kiefer <FredKiefer>
Group Member
Mon 28 Feb 2005 08:05:49 AM UTC, comment #1: 

It seems back should consider font encoding while loading it
and then alter the behavior of

-(void) _generateGlyphsForRun: (glyph_run_t *)run  at: (unsigned int)pos

to match glyph by index (in case that unicode matching paradigm actually
work for standard fonts - I think the submitter test this on iso8859-11,
a free Thai font) Though, alexm told me he will rework this API so it may
not work in future. May be you can provide a link to the font image (Loma font?)

Anonymous
Thu 24 Feb 2005 02:07:16 PM UTC, original submission:  

advancementForGlyph  in Win32 Backend works incorrectly for non-latin characters. It seems like GetCharABCWidthsFloat(hdc, glyph, glyph, &abc) requires glyph index but unicode values were sent.

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by CaS (Posted a comment)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2009-01-30 CaS Open/ClosedIn Test Closed
    2007-02-01 CaS StatusNone Fixed
        Open/ClosedOpen In Test

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code