--- ray/src/px/x11image.c	1991/05/02 17:24:54	1.17
+++ ray/src/px/x11image.c	1991/05/03 13:28:39	1.22
@@ -37,7 +37,7 @@ static char SCCSid[] = "$SunId$ LBL";
 
 #define  BORWIDTH	5		/* border width */
 
-#define  ICONSIZ	80		/* maximum icon dimension */
+#define  ICONSIZ	(8*10)		/* maximum icon dimension (even 8) */
 
 #define  ourscreen	DefaultScreen(thedisplay)
 #define  ourblack	BlackPixel(thedisplay,ourscreen)
@@ -92,7 +92,7 @@ struct {
 
 char  *geometry = NULL;			/* geometry specification */
 
-char  icondata[(ICONSIZ+7)/8*ICONSIZ];	/* icon bitmap data */
+char  icondata[ICONSIZ*ICONSIZ/8];	/* icon bitmap data */
 int  iconwidth = 0, iconheight = 0;
 
 char  *progname;
@@ -246,9 +246,9 @@ init()			/* get data and open window */
 	XSetFont(thedisplay, ourgc, fontid);
 	revgc = XCreateGC(thedisplay, wind, 0, 0);
 	XSetFunction(thedisplay, revgc, GXinvert);
-	XStoreName(thedisplay, wind, fname == NULL ? progname : fname);
 	XDefineCursor(thedisplay, wind, XCreateFontCursor(thedisplay, 
 			XC_diamond_cross));
+	XStoreName(thedisplay, wind, fname == NULL ? progname : fname);
 	if (geometry != NULL) {
 		bzero((char *)&oursizhints, sizeof(oursizhints));
 		i = XParseGeometry(geometry, &oursizhints.x, &oursizhints.y,
@@ -652,13 +652,14 @@ add2icon(y, scan)		/* add a scanline to our icon data 
 int  y;
 COLR  *scan;
 {
-	static char  *dp = NULL;
 	static short  cerr[ICONSIZ];
+	static int  ynext;
+	static char  *dp;
 	register int  err;
-	register int	x, xi;
+	register int	x, ti;
 
 	if (iconheight == 0) {		/* initialize */
-		if (xmax < ICONSIZ && ymax < ICONSIZ) {
+		if (xmax <= ICONSIZ && ymax <= ICONSIZ) {
 			iconwidth = xmax;
 			iconheight = ymax;
 		} else if (xmax > ymax) {
@@ -668,26 +669,24 @@ COLR  *scan;
 			iconwidth = ICONSIZ*xmax/ymax;
 			iconheight = ICONSIZ;
 		}
+		ynext = 0;
 		dp = icondata - 1;
 	}
-	if (dp == NULL)			/* done already */
+	if (y < ynext*ymax/iconheight)	/* skip this one */
 		return;
-	if (y % (ymax/iconheight))	/* skip this one */
-		return;
 	err = 0;
 	for (x = 0; x < iconwidth; x++) {
 		if (!(x&7))
 			*++dp = 0;
-		xi = x*xmax/iconwidth;
-		err += normbright(scan[xi]) + cerr[x];
+		ti = x*xmax/iconwidth;
+		err += normbright(scan[ti]) + cerr[x];
 		if (err > 127)
 			err -= 255;
 		else
 			*dp |= 1<<(x&07);
 		cerr[x] = err >>= 1;
 	}
-	if (y >= ymax-ymax/iconheight)	/* all done */
-		dp = NULL;
+	ynext++;
 }