diff options
| author | Corran Webster <cwebster@unital.dev> | 2024-11-14 10:15:10 +0000 |
|---|---|---|
| committer | Damien George <damien@micropython.org> | 2024-11-28 22:56:44 +1100 |
| commit | e70048cf5934bf3db65ec7a1b6490635716282f5 (patch) | |
| tree | 64cd862fb6c94c129b603c995ac2e4be9e502f93 /py/objfloat.c | |
| parent | 154d1419659795aa6785778e4ddd6f23f09f2861 (diff) | |
| download | micropython-e70048cf5934bf3db65ec7a1b6490635716282f5.tar.gz micropython-e70048cf5934bf3db65ec7a1b6490635716282f5.zip | |
extmod/modframebuf: Fix 0 radius bug in FrameBuffer.ellipse.
This fixes a bug in FrameBuffer.ellipse where it goes into an infinite loop
if both radii are 0.
This fixes the bug with a simple pre-check to see if both radii are 0, and
in that case sets a single pixel at the center. This is consistent with the
behaviour of the method when called with just one of the radii set to 0,
where it will draw a horizontal or vertical line of 1 pixel width.
The pixel is set with setpixel_checked so it should handle out-of-bounds
drawing correctly.
This fix also includes three new tests: one for the default behaviour, one
for drawing out-of-bounds, and one for when the sector mask is 0.
Fixes issue #16053.
Signed-off-by: Corran Webster <cwebster@unital.dev>
Diffstat (limited to 'py/objfloat.c')
0 files changed, 0 insertions, 0 deletions