On 2/29/20 10:31 AM, Sven Schnelle wrote:
On Tue, Feb 18, 2020 at 11:39:24AM -0800, Richard Henderson wrote:
From: Philippe Mathieu-Daudé <f4...@amsat.org>
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Message-Id: <20200214001303.12873-5-f4...@amsat.org>
Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
---
hw/display/artist.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/hw/display/artist.c b/hw/display/artist.c
index 97c811b35e..5492079116 100644
--- a/hw/display/artist.c
+++ b/hw/display/artist.c
@@ -579,6 +579,9 @@ static void draw_line(ARTISTState *s, int x1, int y1, int
x2, int y2,
} else {
dy = y1 - y2;
}
+ if (!dx || !dy) {
+ return;
+ }
c1 = false;
if (dy > dx) {
--
2.20.1
I noticed that this change causes 1px wide lines to disappear. So i would
propose
to revert that change.
Maybe "if (!dx && !dy) { return; }"?
Regards
Sven
