bugGnash - The GNU Flash player - Bugs: bug #22463, text rendered at wrong size

 
 

bug #22463: text rendered at wrong size

Submitter:  Sandro Santilli <strk>
Submitted:  Mon 03 Mar 2008 12:21:54 PM UTC
   
 
Category:  None Severity:  3 - Normal
Release:  None Status:  Fixed
Privacy:  Public Assigned to:  bwy
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Wed 08 Jul 2009 04:11:08 PM UTC, comment #11: 

I confirm size is good now (and width isn't)

Sandro Santilli <strk>
Group Member
Wed 08 Jul 2009 03:56:00 PM UTC, comment #10: 

The size should be correct now. Device fonts are scaled in both directions using the y scale value.

The width of the textfield should be scaled according to the x scale, but that's not yet fixed (needs some changes to TextField.cpp).

The x position of some of the text is still slightly wrong in the two examples in comment #1, but I haven't worked out why. The test misc-ming.all/DeviceFontTest doesn't reproduce this problem.

Benjamin Wolsey <bwy>
Group Member
Sun 09 Nov 2008 02:50:51 PM UTC, comment #9: 

One problem is that we scale the font according using the TextField character's matrix. We shouldn't do this, at least for device fonts.

Benjamin Wolsey <bwy>
Group Member
  Spam posted by anonymous
Fri 14 Mar 2008 12:33:35 PM UTC, comment #7: 

In gnash, device fonts create glyphs on demand.

Sandro Santilli <strk>
Group Member
Tue 11 Mar 2008 01:00:56 PM UTC, comment #6: 

What am I doing? It does use those two device fonts; but the Arial font (ID: 8) has 49 glyphs, each with an associated shape. Is that normal for a device font?


Benjamin Wolsey <bwy>
Group Member
Tue 11 Mar 2008 12:49:56 PM UTC, comment #5: 

That should read "doesn't use device fonts".

Benjamin Wolsey <bwy>
Group Member
Tue 11 Mar 2008 12:35:50 PM UTC, comment #4: 

It does use device fonts: Arial and Symbol (this latter presumably just for the theta sign).

From debug log:

17810] 13:32:41: DEBUG: Advance value for glyph '4' is 284.5 (horiAdvance:569, scale:0.5)
17810] 13:33:58: DEBUG: Advance value for glyph '5' is 569.5 (horiAdvance:1139, scale:0.5)

Benjamin Wolsey <bwy>
Group Member
Thu 06 Mar 2008 09:18:29 PM UTC, comment #3: 

Tried the PP_COMPATIBLE_DEVICE_FONT_HANDLING, didn't work.
Looking at the code that macro only influences color, not size.
To be researched:

1. if the movies deal with device rather then embedded fonts.
2. if device fonts are supposed to be scaled.

Sandro Santilli <strk>
Group Member
Thu 06 Mar 2008 08:41:38 PM UTC, comment #2: 

note to self (or to whoever wants to try that):
try uncommenting the PP_COMPATIBLE_DEVICE_FONT_HANDLING
define in server/edit_text_character.cpp

Sandro Santilli <strk>
Group Member
Mon 03 Mar 2008 12:39:35 PM UTC, comment #1: 
Sandro Santilli <strk>
Group Member
Mon 03 Mar 2008 12:21:54 PM UTC, original submission:  

http://www.upscale.utoronto.ca/GeneralInterest/Harrison/Flash/ClassMechanics/Projectile/Projectile.swf
version: SWF6
md5: 92458afd64496f8089b843e81317b0c9

The dynamic text value of y0 and t? is disaplayed in a weird
way (bigger then expected, and with wrong aspect ratio).
Could be one of those cases in which an application relies on
the NON transformation of device fonts (didn't test if it's device font btw)

Sandro Santilli <strk>
Group Member

 

(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 bjacques (Updated the item)
  • -email is unavailable- added by bwy (Posted a comment)
  • -email is unavailable- added by strk (Submitted the item)
  •  

    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 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-07-08 bjacques StatusReady For Test Fixed
        Assigned toNone bwy
        Open/ClosedOpen Closed
    2009-07-08 bwy StatusConfirmed Ready For Test
    2008-09-17 strk StatusNone Confirmed

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code