wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 380484. Don't divide by zero if a font has a really small maxAdvance. r=vlad

Browse Source
This commit is contained in:
roc+@cs.cmu.edu 2007-05-20 18:56:10 -07:00
parent 1c9b59c045
commit 25a38a1d57
1 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
gfx/thebes/src/gfxAtsuiFonts.cpp
View File

@ -476,9 +476,17 @@ FindTextRunSegmentLength(gfxTextRun *aTextRun, PRUint32 aOffset, PRUint32 aMaxLe
PRUint32
gfxAtsuiFontGroup::GuessMaximumStringLength()
{
gfxFloat maxAdvance = GetFontAt(0)->GetMetrics().maxAdvance;
// ATSUI can't handle offsets of more than 32K pixels
PRUint32 chars = 0x7FFF/PRUint32(maxAdvance);
// ATSUI can't handle offsets of more than 32K pixels. This function
// guesses a string length that ATSUI will be able to handle. We want to
// get the right answer most of the time, but if we're wrong in either
// direction, we won't break things: if we guess too large, our glyph
// processing will detect ATSUI's failure and retry with a smaller limit.
// If we guess too small, we'll just break the string into more pieces
// than we strictly needed to.
// The basic calculation is just 32k pixels divided by the font max-advance,
// but we need to be a bit careful to avoid math errors.
PRUint32 maxAdvance = PRUint32(GetFontAt(0)->GetMetrics().maxAdvance);
PRUint32 chars = 0x7FFF/PR_MAX(1, maxAdvance);
return PR_MAX(1, chars);
}