custom texture binding for Reflective to avoid double free, and race …

…condition fix in appending draw commands for paint example

ReflectiveBoundTexture.go

@@ -7,6 +7,7 @@ import (
 	"image/color"
 	"sync"
 
+	"github.com/AllenDang/cimgui-go/backend"
 	"github.com/AllenDang/cimgui-go/imgui"
 )
 
@@ -193,10 +194,16 @@ func (i *ReflectiveBoundTexture) unbind() {
 }
 
 // bind creates a new texture from the RGBA surface and assigns it to the ReflectiveBoundTexture.
+// note it bypasses normal texture management up to cimgui-go to avoid double free from finalizers.
 func (i *ReflectiveBoundTexture) bind() {
-	NewTextureFromRgba(i.Surface, func(tex *Texture) {
-		i.tex = tex
-	})
+	img := ImageToRgba(i.Surface)
+	i.tex = &Texture{
+		&backend.Texture{
+			ID:     backend.TextureManager(Context.backend).CreateTextureRgba(img, img.Bounds().Dx(), img.Bounds().Dy()),
+			Width:  img.Bounds().Dx(),
+			Height: img.Bounds().Dy(),
+		},
+	}
 }
 
 // GetSurfaceWidth returns the width of the RGBA surface.

examples/paint/canvas.go

@@ -4,6 +4,7 @@ import (
 	"fmt"
 	"image"
 	"image/color"
+	"sync"
 
 	"github.com/AllenDang/cimgui-go/imgui"
 
@@ -57,11 +58,13 @@ type Canvas struct {
 
 	// inited is a boolean flag indicating whether the canvas has been initialized.
 	inited bool
+
+	appendMu sync.Mutex
 }
 
-// GetDrawCommands returns a slice of drawCommand starting from the specified index.
+// getDrawCommands returns a slice of drawCommand starting from the specified index.
 // It allows retrieval of draw commands that have been added since a given point in time.
-func (c *Canvas) GetDrawCommands(sinceIndex int) []DrawCommand {
+func (c *Canvas) getDrawCommands(sinceIndex int) []DrawCommand {
 	return c.DrawCommands[sinceIndex:]
 }
 
@@ -79,6 +82,8 @@ func (c *Canvas) PushImageToBackend(commit bool) error {
 // AppendDrawCommands adds a slice of drawCommand to the canvas's existing draw commands.
 // It appends the provided commands to the DrawCommands slice.
 func (c *Canvas) AppendDrawCommands(cmds *[]DrawCommand) {
+	c.appendMu.Lock()
+	defer c.appendMu.Unlock()
 	c.DrawCommands = append(c.DrawCommands, *cmds...)
 }
 
@@ -104,6 +109,8 @@ func (c *Canvas) Compute() {
 		return
 	}
 
+	c.appendMu.Lock()
+
 	// Return if there are no draw commands to process
 	if len(c.DrawCommands) < 1 {
 		return
@@ -115,7 +122,9 @@ func (c *Canvas) Compute() {
 	}
 
 	// Get the new draw commands that need to be processed
-	draws := c.GetDrawCommands(c.LastComputedLen)
+	draws := c.getDrawCommands(c.LastComputedLen)
+	c.appendMu.Unlock()
+
 	for _, r := range draws {
 		switch r.Tool {
 		case 0: