Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#7045 - Fix bug in font.c

Attached to Project: Rockbox
Opened by takka (tfact) - Tuesday, 17 April 2007, 18:08 GMT+2
Last edited by Steve Bavin (pondlife) - Tuesday, 17 April 2007, 18:55 GMT+2
Task Type Patches
Category LCD
Status Closed
Assigned To No-one
Player Type All players
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

bug fix
Font breaks when MAX_FONT_SIZE is 0xFFFF or more.

   font.patch (0.8 KiB) 
 firmware/font.c |   14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

This task depends upon

Closed by  Paul Louden (Llorean)
Tuesday, 17 April 2007, 20:22 GMT+2
Reason for closing:  Rejected
Additional comments about closing:  This fixes a \"bug\" that only exists if the user changes the value of a #define beyond its expected range (65535) and so is not suitable for inclusion in SVN (adds code that serves no purpose except in user-modified builds)
Comment by Paul Louden (Llorean) - Tuesday, 17 April 2007, 19:59 GMT+2
How is this a bug, if MAX_FONT_SIZE is defined to be less than that?
Comment by takka (tfact) - Tuesday, 17 April 2007, 20:08 GMT+2
Font not breaks when MAX_FONT_SIZE is defined to be less than that.

The display quickens when unicode font is used when the MAX_FONT_SIZE is enlarged.

(ex. unifont.fnt is 1.3MB)
Comment by Paul Louden (Llorean) - Tuesday, 17 April 2007, 20:09 GMT+2
But MAX_FONT_SIZE does not need to be enlarged. The whole point of the glyph cache is so that the whole font doesn't get loaded.

So, to be clear: This is only a bug if the User modifies MAX_FONT_SIZE, right?
Comment by takka (tfact) - Tuesday, 17 April 2007, 20:16 GMT+2
Yes.

Loading...