X-Git-Url: https://git.gag.com/?a=blobdiff_plain;f=src%2Fdraw%2Fao_line.c;h=832a916c76aed9dbe1042f7a024febe071ae93c6;hb=4fa489276826e33fa5e21d0a94a8ed3e29ebcfd0;hp=742b3aa3ec1cc46eca23f0aa634383c768acd74e;hpb=684741765117611b7d666efbdfafd87c6199752c;p=fw%2Faltos

diff --git a/src/draw/ao_line.c b/src/draw/ao_line.c
index 742b3aa3..832a916c 100644
--- a/src/draw/ao_line.c
+++ b/src/draw/ao_line.c
@@ -12,7 +12,6 @@
  * General Public License for more details.
  */
 
-#include "ao.h"
 #include "ao_draw.h"
 #include "ao_draw_int.h"
 
@@ -55,8 +54,9 @@
  *	adjust_x = e / e1;
  */
 
-
-
+#ifdef VALIDATE
+#include <stdio.h>
+#endif
 
 static void
 ao_bres(const struct ao_bitmap	*dst_bitmap,
@@ -80,18 +80,31 @@ ao_bres(const struct ao_bitmap	*dst_bitmap,
 	if (signdx < 0)
 		mask0 = ao_right(1, AO_UNIT - 1);
 
-	if (signdy < 0)
-		stride = -stride;
-
 	dst = dst + y1 * stride + (x1 >> AO_SHIFT);
 	mask = ao_right(1, x1 & AO_MASK);
 
+	if (signdy < 0)
+		stride = -stride;
+
 	while (len--) {
 		/* clip each point */
 
+#ifdef VALIDATE
+		if (x1 < 0 || dst_bitmap->width <= x1) {
+			printf("bad line x %d\n", x1);
+			return;
+		}
+		if (y1 < 0 || dst_bitmap->height <= y1) {
+			printf("bad line y %d\n", y1);
+			return;
+		}
+#endif
 		*dst = ao_do_mask_rrop(*dst, and, xor, mask);
 
 		if (axis == X_AXIS) {
+#ifdef VALIDATE
+			x1 += signdx;
+#endif
 			if (signdx < 0)
 				mask = ao_left(mask, 1);
 			else
@@ -102,13 +115,22 @@ ao_bres(const struct ao_bitmap	*dst_bitmap,
 			}
 			e += e1;
 			if (e >= 0) {
+#ifdef VALIDATE
+				y1 += signdy;
+#endif
 				dst += stride;
 				e += e3;
 			}
 		} else {
+#ifdef VALIDATE
+			y1 += signdy;
+#endif
 			dst += stride;
 			e += e1;
 			if (e >= 0) {
+#ifdef VALIDATE
+				x1 += signdx;
+#endif
 				if (signdx < 0)
 					mask = ao_left(mask, 1);
 				else
@@ -143,12 +165,12 @@ struct ao_cbox {
 /* -b <= a, so we need to make a bigger */
 static int16_t
 div_ceil(int32_t a, int16_t b) {
-	return (a + b + b - 1) / b - 1;
+	return (int16_t) ((a + b + b - 1) / b - 1);
 }
 
 static int16_t
 div_floor_plus_one(int32_t a, int16_t b) {
-	return (a + b) / b;
+	return (int16_t) ((a + b) / b);
 }
 
 static int8_t
@@ -203,10 +225,10 @@ ao_clip_line(struct ao_cc *c, struct ao_cbox *b)
 		adjust_minor = adj_min;
 	}
 
-	c->e += adjust_major * c->e1 + adjust_minor * c->e3;
+	c->e = (int16_t) (c->e + adjust_major * c->e1 + adjust_minor * c->e3);
 
-	c->major += c->sign_major * adjust_major;
-	c->minor += c->sign_minor * adjust_minor;
+	c->major = (int16_t) (c->major + c->sign_major * adjust_major);
+	c->minor = (int16_t) (c->minor + c->sign_minor * adjust_minor);
 
 	return true;
 }
@@ -240,7 +262,7 @@ ao_line(const struct ao_bitmap	*dst,
 	if (adx > ady) {
 		axis = X_AXIS;
 		e1 = ady << 1;
-		e2 = e1 - (adx << 1);
+		e2 = e1 - (int16_t) (adx << 1);
 		e = e1 - adx;
 
 		clip_1.major = x1;
@@ -257,7 +279,7 @@ ao_line(const struct ao_bitmap	*dst,
 	} else {
 		axis = Y_AXIS;
 		e1 = adx << 1;
-		e2 = e1 - (ady << 1);
+		e2 = e1 - (int16_t) (ady << 1);
 		e = e1 - ady;
 
 		clip_1.major = y1;
@@ -290,7 +312,7 @@ ao_line(const struct ao_bitmap	*dst,
 	if (!ao_clip_line(&clip_2, &cbox))
 		return;
 
-	len = clip_1.sign_major * (clip_2.major - clip_1.major) + clip_2.first;
+	len = (int16_t) (clip_1.sign_major * (clip_2.major - clip_1.major) + clip_2.first);
 
 	if (len <= 0)
 		return;