added midpt-ellipse

[Toastee0]

Draws elipses. midptellipse(int x radius, int y radius, Point)
Re-try, after clean rebase.

[Gadgetoid]

I think this will explode since all surface manipulation must now go via set_pixel for a single pixel or set_pixel_span for a horizontal span. The actual framebuffer is now void * and isn't guaranteed to be in any particular format or layout.

[Toastee0]

I'll rewrite it using set pixel, and a check bounds function.

…

On Tue, Jul 5, 2022 at 4:27 AM Philip Howard ***@***.***> wrote: I think this will explode since all surface manipulation must now go via set_pixel for a single pixel or set_pixel_span for a horizontal span. The actual framebuffer is now void * and isn't guaranteed to be in any particular format or layout. — Reply to this email directly, view it on GitHub <#411 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ANH4DLXDD4FEOI7R3OTVYHTVSPWWVANCNFSM5Z6ZMUZA> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[Gadgetoid]

This has been languishing for a while, any interest in bringing it kicking and screaming into 2023 PicoGraphics?

[Gadgetoid]

Note to self that adding Ellipse to #783 might be a good idea.

Closing this for now, feel free to re-open if you decide to tackle it!