internal/ui, internal/mipmap: refactoring: replace At with ReadPixels

Updates #1995
This commit is contained in:
Hajime Hoshi 2022-08-06 00:27:05 +09:00
parent 6c22f3f1a8
commit 6d87be7169
4 changed files with 13 additions and 15 deletions

View File

@ -819,7 +819,9 @@ func (i *Image) at(x, y int) (r, g, b, a byte) {
if c, ok := i.setVerticesCache[[2]int{x, y}]; ok {
return c[0], c[1], c[2], c[3]
}
return i.image.At(x, y)
var pix [4]byte
i.image.ReadPixels(pix[:], x, y, 1, 1)
return pix[0], pix[1], pix[2], pix[3]
}
// Set sets the color at (x, y).

View File

@ -54,12 +54,8 @@ func (m *Mipmap) ReplacePixels(pix []byte, x, y, width, height int) {
m.disposeMipmaps()
}
func (m *Mipmap) At(graphicsDriver graphicsdriver.Graphics, x, y int) (r, g, b, a byte, err error) {
var pix [4]byte
if err := m.orig.ReadPixels(graphicsDriver, pix[:], x, y, 1, 1); err != nil {
return 0, 0, 0, 0, err
}
return pix[0], pix[1], pix[2], pix[3], nil
func (m *Mipmap) ReadPixels(graphicsDriver graphicsdriver.Graphics, pixels []byte, x, y, width, height int) error {
return m.orig.ReadPixels(graphicsDriver, pixels, x, y, width, height)
}
func (m *Mipmap) DrawTriangles(srcs [graphics.ShaderImageCount]*Mipmap, vertices []float32, indices []uint16, colorm affine.ColorM, mode graphicsdriver.CompositeMode, filter graphicsdriver.Filter, address graphicsdriver.Address, dstRegion, srcRegion graphicsdriver.Region, subimageOffsets [graphics.ShaderImageCount - 1][2]float32, shader *Shader, uniforms [][]float32, evenOdd bool, canSkipMipmap bool) {

View File

@ -75,21 +75,21 @@ func (i *Image) ReplacePixels(pix []byte, x, y, width, height int) {
i.mipmap.ReplacePixels(pix, x, y, width, height)
}
func (i *Image) At(x, y int) (r, g, b, a byte) {
func (i *Image) ReadPixels(pixels []byte, x, y, width, height int) {
// Check the error existence and avoid unnecessary calls.
if theGlobalState.error() != nil {
return 0, 0, 0, 0
return
}
r, g, b, a, err := theUI.imageAt(i.mipmap, x, y)
if err != nil {
if err := theUI.readPixels(i.mipmap, pixels, x, y, width, height); err != nil {
if panicOnErrorOnReadingPixels {
panic(err)
}
theGlobalState.setError(err)
return 0, 0, 0, 0
for i := range pixels {
pixels[i] = 0
}
}
return r, g, b, a
}
func (i *Image) DumpScreenshot(name string, blackbg bool) error {

View File

@ -91,8 +91,8 @@ func Get() *UserInterface {
return theUI
}
func (u *UserInterface) imageAt(mipmap *mipmap.Mipmap, x, y int) (r, g, b, a byte, err error) {
return mipmap.At(u.graphicsDriver, x, y)
func (u *UserInterface) readPixels(mipmap *mipmap.Mipmap, pixels []byte, x, y, width, height int) error {
return mipmap.ReadPixels(u.graphicsDriver, pixels, x, y, width, height)
}
func (u *UserInterface) dumpScreenshot(mipmap *mipmap.Mipmap, name string, blackbg bool) error {