Skip to content

Commit 313cfe0

Browse files
stefanhallerstefanhaller
committed
Store TextStyle in Pipe struct as pointer
The instances are held by the AuthorStyle cache.
1 parent b714910 commit 313cfe0

7 files changed

Lines changed: 97 additions & 96 deletions

File tree

‎pkg/gui/presentation/authors/authors.go‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ type authorNameCacheKey struct {
2121
var (
2222
authorInitialCache = make(map[string]string)
2323
authorNameCache = make(map[authorNameCacheKey]string)
24-
authorStyleCache = make(map[string]style.TextStyle)
24+
authorStyleCache = make(map[string]*style.TextStyle)
2525
)
2626

2727
const authorNameWildcard = "*"
@@ -73,7 +73,7 @@ func AuthorWithLength(authorName string, length int) string {
7373
return LongAuthor(authorName, length)
7474
}
7575

76-
func AuthorStyle(authorName string) style.TextStyle {
76+
func AuthorStyle(authorName string) *style.TextStyle {
7777
if value, ok := authorStyleCache[authorName]; ok {
7878
return value
7979
}
@@ -85,9 +85,9 @@ func AuthorStyle(authorName string) style.TextStyle {
8585

8686
value := trueColorStyle(authorName)
8787

88-
authorStyleCache[authorName] = value
88+
authorStyleCache[authorName] = &value
8989

90-
return value
90+
return &value
9191
}
9292

9393
func trueColorStyle(str string) style.TextStyle {

‎pkg/gui/presentation/branches.go‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ import (
2020
)
2121

2222
type colorMatcher struct {
23-
patterns map[string]style.TextStyle
23+
patterns map[string]*style.TextStyle
2424
isRegex bool // NOTE: this value is needed only until the deprecated branchColors config is removed and only regex color patterns are used
2525
}
2626

@@ -142,14 +142,14 @@ func (m *colorMatcher) match(name string) (*style.TextStyle, bool) {
142142
if m.isRegex {
143143
for pattern, style := range m.patterns {
144144
if matched, _ := regexp.MatchString(pattern, name); matched {
145-
return &style, true
145+
return style, true
146146
}
147147
}
148148
} else {
149149
// old behavior using the deprecated branchColors behavior matching on branch type
150150
branchType := strings.Split(name, "/")[0]
151151
if value, ok := m.patterns[branchType]; ok {
152-
return &value, true
152+
return value, true
153153
}
154154
}
155155

‎pkg/gui/presentation/commits.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,7 @@ func loadPipesets(hashPool *utils.StringPool, commits []*models.Commit) [][]grap
259259
if !ok {
260260
// pipe sets are unique to a commit head. and a commit count. Sometimes we haven't loaded everything for that.
261261
// so let's just cache it based on that.
262-
getStyle := func(commit *models.Commit) style.TextStyle {
262+
getStyle := func(commit *models.Commit) *style.TextStyle {
263263
return authors.AuthorStyle(commit.AuthorName)
264264
}
265265
pipeSets = graph.GetPipeSets(hashPool, commits, getStyle)

‎pkg/gui/presentation/graph/cell.go‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ type Cell struct {
2525
up, down, left, right bool
2626
cellType cellType
2727
rightStyle *style.TextStyle
28-
style style.TextStyle
28+
style *style.TextStyle
2929
}
3030

3131
func (cell *Cell) render(writer io.StringWriter) {
@@ -44,7 +44,7 @@ func (cell *Cell) render(writer io.StringWriter) {
4444

4545
var rightStyle *style.TextStyle
4646
if cell.rightStyle == nil {
47-
rightStyle = &cell.style
47+
rightStyle = cell.style
4848
} else {
4949
rightStyle = cell.rightStyle
5050
}
@@ -59,7 +59,7 @@ func (cell *Cell) render(writer io.StringWriter) {
5959
styledSecondChar = cachedSprint(*rightStyle, second)
6060
}
6161

62-
_, _ = writer.WriteString(cachedSprint(cell.style, adjustedFirst))
62+
_, _ = writer.WriteString(cachedSprint(*cell.style, adjustedFirst))
6363
_, _ = writer.WriteString(styledSecondChar)
6464
}
6565

@@ -104,19 +104,19 @@ func (cell *Cell) reset() {
104104
cell.right = false
105105
}
106106

107-
func (cell *Cell) setUp(style style.TextStyle) *Cell {
107+
func (cell *Cell) setUp(style *style.TextStyle) *Cell {
108108
cell.up = true
109109
cell.style = style
110110
return cell
111111
}
112112

113-
func (cell *Cell) setDown(style style.TextStyle) *Cell {
113+
func (cell *Cell) setDown(style *style.TextStyle) *Cell {
114114
cell.down = true
115115
cell.style = style
116116
return cell
117117
}
118118

119-
func (cell *Cell) setLeft(style style.TextStyle) *Cell {
119+
func (cell *Cell) setLeft(style *style.TextStyle) *Cell {
120120
cell.left = true
121121
if !cell.up && !cell.down {
122122
// vertical trumps left
@@ -126,15 +126,15 @@ func (cell *Cell) setLeft(style style.TextStyle) *Cell {
126126
}
127127

128128
//nolint:unparam
129-
func (cell *Cell) setRight(style style.TextStyle, override bool) *Cell {
129+
func (cell *Cell) setRight(style *style.TextStyle, override bool) *Cell {
130130
cell.right = true
131131
if cell.rightStyle == nil || override {
132-
cell.rightStyle = &style
132+
cell.rightStyle = style
133133
}
134134
return cell
135135
}
136136

137-
func (cell *Cell) setStyle(style style.TextStyle) *Cell {
137+
func (cell *Cell) setStyle(style *style.TextStyle) *Cell {
138138
cell.style = style
139139
return cell
140140
}

‎pkg/gui/presentation/graph/graph.go‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ type Pipe struct {
2828
fromHash *string
2929
toHash *string
3030
kind PipeKind
31-
style style.TextStyle
31+
style *style.TextStyle
3232
}
3333

3434
var highlightStyle = style.FgLightWhite.SetBold()
@@ -41,7 +41,7 @@ func (self Pipe) right() int {
4141
return max(self.fromPos, self.toPos)
4242
}
4343

44-
func RenderCommitGraph(hashPool *utils.StringPool, commits []*models.Commit, selectedCommitHash *string, getStyle func(c *models.Commit) style.TextStyle) []string {
44+
func RenderCommitGraph(hashPool *utils.StringPool, commits []*models.Commit, selectedCommitHash *string, getStyle func(c *models.Commit) *style.TextStyle) []string {
4545
pipeSets := GetPipeSets(hashPool, commits, getStyle)
4646
if len(pipeSets) == 0 {
4747
return nil
@@ -52,12 +52,12 @@ func RenderCommitGraph(hashPool *utils.StringPool, commits []*models.Commit, sel
5252
return lines
5353
}
5454

55-
func GetPipeSets(hashPool *utils.StringPool, commits []*models.Commit, getStyle func(c *models.Commit) style.TextStyle) [][]Pipe {
55+
func GetPipeSets(hashPool *utils.StringPool, commits []*models.Commit, getStyle func(c *models.Commit) *style.TextStyle) [][]Pipe {
5656
if len(commits) == 0 {
5757
return nil
5858
}
5959

60-
pipes := []Pipe{{fromPos: 0, toPos: 0, fromHash: hashPool.Add("START"), toHash: commits[0].Hash, kind: STARTS, style: style.FgDefault}}
60+
pipes := []Pipe{{fromPos: 0, toPos: 0, fromHash: hashPool.Add("START"), toHash: commits[0].Hash, kind: STARTS, style: &style.FgDefault}}
6161

6262
return lo.Map(commits, func(commit *models.Commit, _ int) []Pipe {
6363
pipes = getNextPipes(hashPool, pipes, commit, getStyle)
@@ -102,7 +102,7 @@ func RenderAux(pipeSets [][]Pipe, commits []*models.Commit, selectedCommitHash *
102102
return lo.Flatten(chunks)
103103
}
104104

105-
func getNextPipes(hashPool *utils.StringPool, prevPipes []Pipe, commit *models.Commit, getStyle func(c *models.Commit) style.TextStyle) []Pipe {
105+
func getNextPipes(hashPool *utils.StringPool, prevPipes []Pipe, commit *models.Commit, getStyle func(c *models.Commit) *style.TextStyle) []Pipe {
106106
maxPos := 0
107107
for _, pipe := range prevPipes {
108108
if pipe.toPos > maxPos {
@@ -294,10 +294,10 @@ func renderPipeSet(
294294
isMerge := startCount > 1
295295

296296
cells := lo.Map(lo.Range(maxPos+1), func(i int, _ int) *Cell {
297-
return &Cell{cellType: CONNECTION, style: style.FgDefault}
297+
return &Cell{cellType: CONNECTION, style: &style.FgDefault}
298298
})
299299

300-
renderPipe := func(pipe *Pipe, style style.TextStyle, overrideRightStyle bool) {
300+
renderPipe := func(pipe *Pipe, style *style.TextStyle, overrideRightStyle bool) {
301301
left := pipe.left()
302302
right := pipe.right()
303303

@@ -353,9 +353,9 @@ func renderPipeSet(
353353
}
354354
}
355355
for _, pipe := range selectedPipes {
356-
renderPipe(&pipe, highlightStyle, true)
356+
renderPipe(&pipe, &highlightStyle, true)
357357
if pipe.toPos == commitPos {
358-
cells[pipe.toPos].setStyle(highlightStyle)
358+
cells[pipe.toPos].setStyle(&highlightStyle)
359359
}
360360
}
361361

0 commit comments

Comments
 (0)