lovyan03 / LovyanGFX

SPI LCD graphics library for ESP32 (ESP-IDF/ArduinoESP32) / ESP8266 (ArduinoESP8266) / SAMD51(Seeed ArduinoSAMD51)
Other
1.02k stars 187 forks source link

drawEllipseとfillEllipseの、rxとryの一方だけが0のときの挙動について #544

Closed matsuura-h closed 1 month ago

matsuura-h commented 1 month ago

素晴らしいライブラリの公開ありがとうございます。

Environment ( 実行環境 )

Problem Description ( 問題の内容 )

冒頭のrx, ryがゼロのときの処理ですが、線を引く処理の第3引数についてrx, ryが逆ではないでしょうか? if (ry == 0) { drawFastHLine(x - rx, y, (ry << 2) + 1); return; } if (rx == 0) { drawFastVLine(x, y - ry, (rx << 2) + 1); return; }

本来の意図として、例えばryが0のときは、横幅がrxの2倍+1の横線を表示するほうがよいのでは?と考えています。 ご検討お願いいたします。

lovyan03 commented 1 month ago

@matsuura-h 情報ありがとうございます!これはうっかりミスですね…修正します。 お知らせくださって助かりました。

matsuura-h commented 1 month ago

@lovyan03 ご確認ありがとうございます。 修正いただいたソースも拝見いたしました。 本件、Closeとさせていただきます。