wdvi

commit 375592be35b2e15403d648043f111103cb68d0df
parent 419036170c0a81e461924983616172641a7be745
Author: Kyle Milz <krwmilz@gmail.com>
Date:   Thu,  9 Mar 2023 01:06:42 +0000

use inverted if conditions to return early in put_rule()

Diffstat:
M
dvi-draw.c
 | 
15
+++++++++------

1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/dvi-draw.c b/dvi-draw.c
@@ -103,12 +103,15 @@ static	void	draw_part(struct frame *, double);
 static	void
 put_rule(int x, int y, unsigned int w, unsigned int h)
 {
-	if (x < max_x && x + (int) w >= min_x
-	  && y < max_y && y + (int) h >= min_y) {
-	    if (fg_active != fg_current) do_color_change();
-	    XFillRectangle(DISP, currwin.win, ruleGC,
-		x - currwin.base_x, y - currwin.base_y, w ? w : 1, h ? h : 1);
-	}
+	if (x >= max_x || x + (int) w < min_x ||
+	    y >= max_y || y + (int) h < min_y)
+		return;
+
+	if (fg_active != fg_current)
+		do_color_change();
+	XFillRectangle(DISP, currwin.win, ruleGC,
+	               x - currwin.base_x, y - currwin.base_y,
+	               w ? w : 1, h ? h : 1);
 }
 
 static	void