mth/ebiten
1
0
Fork 0
mirror of https://github.com/hajimehoshi/ebiten.git synced 2024-12-25 11:18:54 +01:00
Code Issues Packages Projects Releases Wiki Activity

text: Remove uniqFace for performance

Browse Source 
Fixes #554. Add comments for #498.
This commit is contained in:
Hajime Hoshi 2018-03-15 23:00:23 +09:00
parent 1b35bbec9a
commit 9092ec083d
1 changed files with 5 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
text/text.go
View File

@ -21,7 +21,6 @@ import (
"image"
"image/color"
"math"
"reflect"
"golang.org/x/image/font"
"golang.org/x/image/math/fixed"
@ -112,25 +111,6 @@ func drawGlyph(dst *ebiten.Image, face font.Face, r rune, x, y fixed.Int26_6, cl
_ = dst.DrawImage(img, op)
}
var (
fontFaces = map[font.Face]struct{}{}
)
func uniqFace(f font.Face) font.Face {
if _, ok := fontFaces[f]; ok {
return f
}
// If the (DeepEqual-ly) same font exists,
// reuse this to avoid to consume a lot of cache (#498).
for key := range fontFaces {
if reflect.DeepEqual(key, f) {
return key
}
}
fontFaces[f] = struct{}{}
return f
}
var (
// Use pointers to avoid copying on browsers.
glyphBoundsCache = map[font.Face]map[rune]*fixed.Rectangle26_6{}
@ -223,6 +203,9 @@ var textM sync.Mutex
// Glyphs used for rendering are cached in least-recently-used way.
// It is OK to call this function with a same text and a same face at every frame in terms of performance.
//
// Be careful that the passed font face is held by this package and is never released.
// This is a known issue (#498).
//
// This function is concurrent-safe.
func Draw(dst *ebiten.Image, text string, face font.Face, x, y int, clr color.Color) {
textM.Lock()
@ -235,9 +218,8 @@ func Draw(dst *ebiten.Image, text string, face font.Face, x, y int, clr color.Co
if prevR >= 0 {
fx += face.Kern(prevR, r)
}
fa := uniqFace(face)
drawGlyph(dst, fa, r, fx, fixed.I(y), clr)
fx += glyphAdvance(fa, r)
drawGlyph(dst, face, r, fx, fixed.I(y), clr)
fx += glyphAdvance(face, r)
prevR = r
}