From 7cc6abeee3b221b76e1e24065f4ae4107b35ff0f Mon Sep 17 00:00:00 2001
From: redstarcoder <oobqvfgq@pokemail.net>
Date: Tue, 25 Oct 2016 21:15:42 -0400
Subject: [PATCH] Made FetchGlyph and Glyph public, made FillGlyph and
 StrokeGlyph methods of Glyph, made Glyph.Path private

---
 draw2dbase/text.go | 74 ++++++++++++++++++++++------------------------
 draw2dgl/gc.go     |  6 ++--
 draw2dimg/ftgc.go  |  6 ++--
 3 files changed, 44 insertions(+), 42 deletions(-)

diff --git a/draw2dbase/text.go b/draw2dbase/text.go
index 53eed21..4283bad 100644
--- a/draw2dbase/text.go
+++ b/draw2dbase/text.go
@@ -2,38 +2,16 @@ package draw2dbase
 
 import "github.com/llgcode/draw2d"
 
-var glyphCache map[string]map[rune]*glyph
+var glyphCache map[string]map[rune]*Glyph
 
 func init() {
-	glyphCache = make(map[string]map[rune]*glyph)
+	glyphCache = make(map[string]map[rune]*Glyph)
 }
 
-// FillGlyph copies a glyph from the cache, copies it to the gc, and fills it
-func FillGlyph(gc draw2d.GraphicContext, x, y float64, fontName string, chr rune) float64 {
-	g := fetchGlyph(gc, fontName, chr)
-	gc.Save()
-	gc.BeginPath()
-	gc.Translate(x, y)
-	gc.Fill(g.Path)
-	gc.Restore()
-	return g.Width
-}
-
-// StrokeGlyph fetches a glyph from the cache, copies it to the gc, and strokes it
-func StrokeGlyph(gc draw2d.GraphicContext, x, y float64, fontName string, chr rune) float64 {
-	g := fetchGlyph(gc, fontName, chr)
-	gc.Save()
-	gc.BeginPath()
-	gc.Translate(x, y)
-	gc.Stroke(g.Path)
-	gc.Restore()
-	return g.Width
-}
-
-// fetchGlyph fetches a glyph from the cache, calling renderGlyph first if it doesn't already exist
-func fetchGlyph(gc draw2d.GraphicContext, fontName string, chr rune) *glyph {
+// FetchGlyph fetches a glyph from the cache, calling renderGlyph first if it doesn't already exist
+func FetchGlyph(gc draw2d.GraphicContext, fontName string, chr rune) *Glyph {
 	if glyphCache[fontName] == nil {
-		glyphCache[fontName] = make(map[rune]*glyph, 60)
+		glyphCache[fontName] = make(map[rune]*Glyph, 60)
 	}
 	if glyphCache[fontName][chr] == nil {
 		glyphCache[fontName][chr] = renderGlyph(gc, fontName, chr)
@@ -41,30 +19,50 @@ func fetchGlyph(gc draw2d.GraphicContext, fontName string, chr rune) *glyph {
 	return glyphCache[fontName][chr].Copy()
 }
 
-// renderGlyph renders a Glyph then caches and returns it
-func renderGlyph(gc draw2d.GraphicContext, fontName string, chr rune) *glyph {
+// renderGlyph renders a glyph then caches and returns it
+func renderGlyph(gc draw2d.GraphicContext, fontName string, chr rune) *Glyph {
 	gc.Save()
 	defer gc.Restore()
 	gc.BeginPath()
 	width := gc.CreateStringPath(string(chr), 0, 0)
 	path := gc.GetPath()
-	return &glyph{
-		Path:  &path,
+	return &Glyph{
+		path:  &path,
 		Width: width,
 	}
 }
 
-// glyph represents a rune which has been converted to a Path and width
-type glyph struct {
-	// Path represents a glyph, it is always at (0, 0)
-	Path *draw2d.Path
+// Glyph represents a rune which has been converted to a Path and width
+type Glyph struct {
+	// path represents a glyph, it is always at (0, 0)
+	path *draw2d.Path
 	// Width of the glyph
 	Width float64
 }
 
-func (g *glyph) Copy() *glyph {
-	return &glyph{
-		Path:  g.Path.Copy(),
+func (g *Glyph) Copy() *Glyph {
+	return &Glyph{
+		path:  g.path.Copy(),
 		Width: g.Width,
 	}
 }
+
+// Fill copies a glyph from the cache, and fills it
+func (g *Glyph) Fill(gc draw2d.GraphicContext, x, y float64) float64 {
+	gc.Save()
+	gc.BeginPath()
+	gc.Translate(x, y)
+	gc.Fill(g.path)
+	gc.Restore()
+	return g.Width
+}
+
+// Stroke fetches a glyph from the cache, and strokes it
+func (g *Glyph) Stroke(gc draw2d.GraphicContext, x, y float64) float64 {
+	gc.Save()
+	gc.BeginPath()
+	gc.Translate(x, y)
+	gc.Stroke(g.path)
+	gc.Restore()
+	return g.Width
+}
diff --git a/draw2dgl/gc.go b/draw2dgl/gc.go
index c08d1f4..ec2da25 100644
--- a/draw2dgl/gc.go
+++ b/draw2dgl/gc.go
@@ -207,7 +207,8 @@ func (gc *GraphicContext) FillStringAt(text string, x, y float64) (width float64
 	xorig := x
 	fontName := gc.GetFontName()
 	for _, r := range text {
-		x += draw2dbase.FillGlyph(gc, x, y, fontName, r)
+		glyph := draw2dbase.FetchGlyph(gc, fontName, r)
+		x += glyph.Fill(gc, x, y)
 	}
 	return x - xorig
 }
@@ -261,7 +262,8 @@ func (gc *GraphicContext) StrokeStringAt(text string, x, y float64) (width float
 	xorig := x
 	fontName := gc.GetFontName()
 	for _, r := range text {
-		x += draw2dbase.StrokeGlyph(gc, x, y, fontName, r)
+		glyph := draw2dbase.FetchGlyph(gc, fontName, r)
+		x += glyph.Stroke(gc, x, y)
 	}
 	return x - xorig
 }
diff --git a/draw2dimg/ftgc.go b/draw2dimg/ftgc.go
index 9401d50..1f0fa4a 100644
--- a/draw2dimg/ftgc.go
+++ b/draw2dimg/ftgc.go
@@ -126,7 +126,8 @@ func (gc *GraphicContext) FillStringAt(text string, x, y float64) (width float64
 	xorig := x
 	fontName := gc.GetFontName()
 	for _, r := range text {
-		x += draw2dbase.FillGlyph(gc, x, y, fontName, r)
+		glyph := draw2dbase.FetchGlyph(gc, fontName, r)
+		x += glyph.Fill(gc, x, y)
 	}
 	return x - xorig
 }
@@ -141,7 +142,8 @@ func (gc *GraphicContext) StrokeStringAt(text string, x, y float64) (width float
 	xorig := x
 	fontName := gc.GetFontName()
 	for _, r := range text {
-		x += draw2dbase.StrokeGlyph(gc, x, y, fontName, r)
+		glyph := draw2dbase.FetchGlyph(gc, fontName, r)
+		x += glyph.Stroke(gc, x, y)
 	}
 	return x - xorig
 }