Some lighter refactoring

draw.go

@@ -6,13 +6,14 @@ import "git.tebibyte.media/sashakoshka/stone"
 
 func redraw () {
 	width, height := application.Size()
-	columns = (width - 20) / 23 + 1
+	state.columns  = (width - 20) / 23 + 1
-	rows    = height / 10 + 1
+	state.rows     = height / 10 + 1
+	state.pageSize = state.columns * state.rows
 
-	monthIter := viewingMonth
+	monthIter := state.viewingMonth
-	xOffset := (width - columns * 23) / 2 + 1
+	xOffset := (width - state.columns * 23) / 2 + 1
-	for y := 0; y < rows; y ++ {
+	for y := 0; y < state.rows; y ++ {
-		for x := 0; x < columns;  x ++ {
+		for x := 0; x < state.columns;  x ++ {
 			drawMonth (
 				x * 23 + xOffset, y * 10,
 				monthIter / 12,
@@ -23,7 +24,7 @@ func redraw () {
 }
 
 func drawMonth (xOffset, yOffset, year int, month time.Month) {
-	current := int(month) - 1 + year * 12 == currentMonth
+	current := int(month) - 1 + year * 12 == state.currentMonth
 
 	bce := year < 0
 	if bce { year *= -1 }

event.go

@@ -1,52 +1,46 @@
 package main
 
+import "sync"
 import "time"
 import "git.tebibyte.media/sashakoshka/stone"
 
-func onScroll (x, y int) {
+var lock sync.Mutex
-	drawMutex.Lock()
-	defer drawMutex.Unlock()
 
-	viewingMonth += y * columns
+func onScroll (x, y int) {
+	lock.Lock()
+	defer lock.Unlock()
+	
+	state.viewingMonth += y * state.columns
 	application.Clear()
 	redraw()
 	application.Draw()
 }
 
 func onPress (button stone.Button, modifiers stone.Modifiers) {
+	lock.Lock()
+	defer lock.Unlock()
+	
 	switch button {
 	case stone.KeyUp:
-		drawMutex.Lock()
+		state.viewingMonth -= state.columns
-		defer drawMutex.Unlock()
-	
-		viewingMonth -= columns
 		application.Clear()
 		redraw()
 		application.Draw()
 		
 	case stone.KeyDown:
-		drawMutex.Lock()
+		state.viewingMonth += state.columns
-		defer drawMutex.Unlock()
-	
-		viewingMonth += columns
 		application.Clear()
 		redraw()
 		application.Draw()
 		
 	case stone.KeyPageUp:
-		drawMutex.Lock()
+		state.viewingMonth -= state.pageSize
-		defer drawMutex.Unlock()
-	
-		viewingMonth -= columns * rows
 		application.Clear()
 		redraw()
 		application.Draw()
 		
 	case stone.KeyPageDown:
-		drawMutex.Lock()
+		state.viewingMonth += state.pageSize
-		defer drawMutex.Unlock()
-		
-		viewingMonth += columns * rows
 		application.Clear()
 		redraw()
 		application.Draw()
@@ -55,11 +49,11 @@ func onPress (button stone.Button, modifiers stone.Modifiers) {
 }
 
 func onStart () {
-	drawMutex.Lock()
+	lock.Lock()
-	defer drawMutex.Unlock()
+	defer lock.Unlock()
 	
-	currentMonth = canonMonth(time.Now())
+	state.currentMonth = canonMonth(time.Now())
-	viewingMonth = currentMonth
+	state.viewingMonth = state.currentMonth
 	redraw()
 	go func () {
 		for {
@@ -70,30 +64,29 @@ func onStart () {
 }
 
 func onResize () {
-	drawMutex.Lock()
+	lock.Lock()
-	defer drawMutex.Unlock()
+	defer lock.Unlock()
 	
 	redraw()
 }
 
 func onTick () {
-	drawMutex.Lock()
+	lock.Lock()
-	defer drawMutex.Unlock()
+	defer lock.Unlock()
 
-	pageSize := columns * rows
 	within :=
-		currentMonth >= viewingMonth &&
+		state.currentMonth >= state.viewingMonth &&
-		currentMonth <  viewingMonth + pageSize
+		state.currentMonth <  state.viewingMonth + state.pageSize
 
 	newMonth := canonMonth(time.Now())
-	if currentMonth != newMonth {
+	if state.currentMonth != newMonth {
-		currentMonth = newMonth
+		state.currentMonth = newMonth
 
 		if within {
-			if currentMonth < viewingMonth {
+			if state.currentMonth < state.viewingMonth {
-				viewingMonth -= pageSize
+				state.viewingMonth -= state.pageSize
-			} else if currentMonth >= viewingMonth + pageSize {
+			} else if state.currentMonth >= state.pageSize {
-				viewingMonth += pageSize
+				state.viewingMonth += state.pageSize
 			}
 		}
 		

main.go

@@ -1,6 +1,5 @@
 package main
 
-import "sync"
 import "bytes"
 import "image"
 import _ "embed"
@@ -13,10 +12,10 @@ var iconBytes []byte
 
 var application = &stone.Application { }
 
-var viewingMonth int
+var state struct {
-var currentMonth int
+	viewingMonth, currentMonth int
-var columns, rows int
+	columns, rows, pageSize int
-var drawMutex sync.Mutex
+}
 
 var shortMonthNames = []string {
 	"",