Merge pull request #3280 from Starbuck5/tiny-gfxdraw-adjustment

ankith26 · web-flow · commit e5e2e5fa9f5d · 2025-01-05T13:29:11.000+05:30
Remove unnecessary lock in gfxdraw
diff --git a/src_c/SDL_gfx/SDL_gfxPrimitives.c b/src_c/SDL_gfx/SDL_gfxPrimitives.c
@@ -4093,7 +4093,7 @@ ellipseColor(SDL_Surface *dst, Sint16 x, Sint16 y, Sint16 rx, Sint16 ry,
                         result |= pixelColorNolock(dst, xmi, ypj, color);
                         result |= pixelColorNolock(dst, xpi, ypj, color);
                         result |= pixelColorNolock(dst, xmi, ymj, color);
-                        result |= pixelColor(dst, xpi, ymj, color);
+                        result |= pixelColorNolock(dst, xpi, ymj, color);
                     }
                     else {
                         result |= pixelColorNolock(dst, xmi, y, color);

Original file line number	Diff line number	Diff line change
`@@ -4093,7 +4093,7 @@ ellipseColor(SDL_Surface *dst, Sint16 x, Sint16 y, Sint16 rx, Sint16 ry,`
`4093`	`4093`	`result \|= pixelColorNolock(dst, xmi, ypj, color);`
`4094`	`4094`	`result \|= pixelColorNolock(dst, xpi, ypj, color);`
`4095`	`4095`	`result \|= pixelColorNolock(dst, xmi, ymj, color);`
`4096`		`- result \|= pixelColor(dst, xpi, ymj, color);`
	`4096`	`+ result \|= pixelColorNolock(dst, xpi, ymj, color);`
`4097`	`4097`	`}`
`4098`	`4098`	`else {`
`4099`	`4099`	`result \|= pixelColorNolock(dst, xmi, y, color);`