added midpt-ellipse

[Toastee0]

trafficstars

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 https://github.com/pimoroni/pimoroni-pico/pull/411#issuecomment-1174769552, 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!