Major changes to code (#65)

Author: mokiat

.github/workflows/go.yml

@@ -17,7 +17,7 @@ jobs:
       - name: Setup Go
         uses: actions/setup-go@v5
         with:
-          go-version: "1.23"
+          go-version: "1.25"
 
       - name: Run Tests
-        run: go run github.com/onsi/ginkgo/v2/ginkgo -r -randomize-all
+        run: go tool ginkgo -r -randomize-all

app/controller.go

@@ -1,5 +1,9 @@
 package app
 
+import (
+	"slices"
+)
+
 // Controller is a mechanism through which the user code can be
 // notified of changes to the application window.
 //
@@ -33,6 +37,12 @@ type Controller interface {
 	// not be propagated to other potential receivers, otherwise return false.
 	OnMouseEvent(window Window, event MouseEvent) bool
 
+	// OnGamepadEvent is called whenever a gamepad event has occurred.
+	//
+	// Return true to indicate that the event has been consumed and should
+	// not be propagated to other potential receivers, otherwise return false.
+	OnGamepadEvent(window Window, event GamepadEvent) bool
+
 	// OnClipboardEvent is called whenever the clipboard content has been
 	// requested and the underlying window has managed to retrieve it.
 	OnClipboardEvent(window Window, event ClipboardEvent) bool
@@ -71,6 +81,8 @@ func (NopController) OnKeyboardEvent(window Window, event KeyboardEvent) bool {
 
 func (NopController) OnMouseEvent(window Window, event MouseEvent) bool { return false }
 
+func (NopController) OnGamepadEvent(window Window, event GamepadEvent) bool { return false }
+
 func (NopController) OnClipboardEvent(window Window, event ClipboardEvent) bool { return false }
 
 func (NopController) OnRender(window Window) {}
@@ -97,67 +109,76 @@ type LayeredController struct {
 }
 
 func (c *LayeredController) OnCreate(window Window) {
-	for i := 0; i < len(c.layers); i++ {
-		c.layers[i].OnCreate(window)
+	for _, layer := range c.layers {
+		layer.OnCreate(window)
 	}
 }
 
 func (c *LayeredController) OnResize(window Window, width, height int) {
-	for i := 0; i < len(c.layers); i++ {
-		c.layers[i].OnResize(window, width, height)
+	for _, layer := range c.layers {
+		layer.OnResize(window, width, height)
 	}
 }
 
 func (c *LayeredController) OnFramebufferResize(window Window, width, height int) {
-	for i := 0; i < len(c.layers); i++ {
-		c.layers[i].OnFramebufferResize(window, width, height)
+	for _, layer := range c.layers {
+		layer.OnFramebufferResize(window, width, height)
 	}
 }
 
 func (c *LayeredController) OnKeyboardEvent(window Window, event KeyboardEvent) bool {
-	for i := len(c.layers) - 1; i >= 0; i-- {
-		if c.layers[i].OnKeyboardEvent(window, event) {
+	for _, layer := range slices.Backward(c.layers) {
+		if layer.OnKeyboardEvent(window, event) {
 			return true
 		}
 	}
 	return false
 }
 
 func (c *LayeredController) OnMouseEvent(window Window, event MouseEvent) bool {
-	for i := len(c.layers) - 1; i >= 0; i-- {
-		if c.layers[i].OnMouseEvent(window, event) {
+	for _, layer := range slices.Backward(c.layers) {
+		if layer.OnMouseEvent(window, event) {
+			return true
+		}
+	}
+	return false
+}
+
+func (c *LayeredController) OnGamepadEvent(window Window, event GamepadEvent) bool {
+	for _, layer := range slices.Backward(c.layers) {
+		if layer.OnGamepadEvent(window, event) {
 			return true
 		}
 	}
 	return false
 }
 
 func (c *LayeredController) OnClipboardEvent(window Window, event ClipboardEvent) bool {
-	for i := len(c.layers) - 1; i >= 0; i-- {
-		if c.layers[i].OnClipboardEvent(window, event) {
+	for _, layer := range slices.Backward(c.layers) {
+		if layer.OnClipboardEvent(window, event) {
 			return true
 		}
 	}
 	return false
 }
 
 func (c *LayeredController) OnRender(window Window) {
-	for i := 0; i < len(c.layers); i++ {
-		c.layers[i].OnRender(window)
+	for _, layer := range c.layers {
+		layer.OnRender(window)
 	}
 }
 
 func (c *LayeredController) OnCloseRequested(window Window) bool {
-	for i := len(c.layers) - 1; i >= 0; i-- {
-		if !c.layers[i].OnCloseRequested(window) {
+	for _, layer := range slices.Backward(c.layers) {
+		if !layer.OnCloseRequested(window) {
 			return false
 		}
 	}
 	return true
 }
 
 func (c *LayeredController) OnDestroy(window Window) {
-	for i := len(c.layers) - 1; i >= 0; i-- {
-		c.layers[i].OnDestroy(window)
+	for _, layer := range slices.Backward(c.layers) {
+		layer.OnDestroy(window)
 	}
 }