internal/buffered: remove unnecessary deferred functions for shaders

internal/atlas/image.go

@@ -393,7 +393,7 @@ func (i *Image) drawTriangles(srcs [graphics.ShaderImageCount]*Image, vertices [
 			vertices[i+2] += oxf
 			vertices[i+3] += oyf
 		}
-		if shader.unit() == shaderir.Texels {
+		if shader.ensureShader().Unit() == shaderir.Texels {
 			sw, sh := srcs[0].backend.restorable.InternalSize()
 			swf, shf := float32(sw), float32(sh)
 			for i := 0; i < n; i += graphics.VertexFloatCount {
@@ -432,7 +432,7 @@ func (i *Image) drawTriangles(srcs [graphics.ShaderImageCount]*Image, vertices [
 		imgs[i] = src.backend.restorable
 	}
 
-	i.backend.restorable.DrawTriangles(imgs, vertices, indices, blend, dstRegion, srcRegions, shader.shader, uniforms, evenOdd)
+	i.backend.restorable.DrawTriangles(imgs, vertices, indices, blend, dstRegion, srcRegions, shader.ensureShader(), uniforms, evenOdd)
 
 	for _, src := range srcs {
 		if src == nil {

internal/atlas/shader.go

@@ -22,22 +22,24 @@ import (
 )
 
 type Shader struct {
+	ir     *shaderir.Program
 	shader *restorable.Shader
 }
 
 func NewShader(ir *shaderir.Program) *Shader {
-	backendsM.Lock()
+	// A shader is initialized lazily, and the lock is not needed.
-	defer backendsM.Unlock()
+	return &Shader{
-
+		ir: ir,
-	s := &Shader{
-		shader: restorable.NewShader(ir),
 	}
-	runtime.SetFinalizer(s, (*Shader).MarkDisposed)
-	return s
 }
 
-func (s *Shader) unit() shaderir.Unit {
+func (s *Shader) ensureShader() *restorable.Shader {
-	return s.shader.Unit()
+	if s.shader != nil {
+		return s.shader
+	}
+	s.shader = restorable.NewShader(s.ir)
+	s.ir = nil
+	return s.shader
 }
 
 // MarkDisposed marks the shader as disposed. The actual operation is deferred.

internal/buffered/image.go

@@ -162,38 +162,12 @@ type Shader struct {
 }
 
 func NewShader(ir *shaderir.Program) *Shader {
-	s := &Shader{}
+	return &Shader{
-	s.initialize(ir)
+		shader: atlas.NewShader(ir),
-	return s
-}
-
-func (s *Shader) initialize(ir *shaderir.Program) {
-	if maybeCanAddDelayedCommand() {
-		if tryAddDelayedCommand(func() {
-			s.initializeImpl(ir)
-		}) {
-			return
-		}
 	}
-	s.initializeImpl(ir)
-}
-
-func (s *Shader) initializeImpl(ir *shaderir.Program) {
-	s.shader = atlas.NewShader(ir)
 }
 
 func (s *Shader) MarkDisposed() {
-	if maybeCanAddDelayedCommand() {
-		if tryAddDelayedCommand(func() {
-			s.markDisposedImpl()
-		}) {
-			return
-		}
-	}
-	s.markDisposedImpl()
-}
-
-func (s *Shader) markDisposedImpl() {
 	s.shader.MarkDisposed()
 	s.shader = nil
 }