--- ray/src/px/xshowtrace.c	2003/02/22 02:07:28	2.7
+++ ray/src/px/xshowtrace.c	2004/03/28 20:33:14	2.10
@@ -1,5 +1,5 @@
 #ifndef lint
-static const char	RCSid[] = "$Id: xshowtrace.c,v 2.7 2003/02/22 02:07:28 greg Exp $";
+static const char	RCSid[] = "$Id: xshowtrace.c,v 2.10 2004/03/28 20:33:14 schorsch Exp $";
 #endif
 /*
  *  Display an image and watch the rays get traced.
@@ -7,9 +7,11 @@ static const char	RCSid[] = "$Id: xshowtrace.c,v 2.7 2
  *	9/21/90	Greg Ward
  */
 
+#include <X11/Xlib.h>
+
 #include "standard.h"
+#include "paths.h"
 #include "view.h"
-#include <X11/Xlib.h>
 
 #define MAXDEPTH	32		/* ridiculous ray tree depth */
 
@@ -43,13 +45,22 @@ struct node {				/* ray tree node */
 
 int	slow = 0;		/* slow trace? */
 
+void mainloop(void);
+static void freetree(struct node	*tp);
+static void tracerays(struct node	*tp);
+static int strtoipt(int	ipt[2], char	*str);
+static void setvec(int	ipt[2]);
+static void vector(int	ip1[2], int	ip2[2]);
 
-main(argc, argv)		/* takes both the octree and the image */
-int	argc;
-char	*argv[];
+
+int
+main(		/* takes both the octree and the image */
+	int	argc,
+	char	*argv[]
+)
 {
 	int	i;
-	char	combuf[256];
+	char	combuf[PATH_MAX];
 
 	progname = argv[0];
 	for (i = 1; i < argc-2; i++)
@@ -78,7 +89,7 @@ char	*argv[];
 		exit(1);
 	}
 					/* build input command */
-	sprintf(combuf, "%s %s | %s", xicom, picture, rtcom);
+	sprintf(combuf, "%s \"%s\" | %s", xicom, picture, rtcom);
 	for ( ; i < argc-1; i++) {
 		strcat(combuf, " ");
 		strcat(combuf, argv[i]);
@@ -94,7 +105,8 @@ char	*argv[];
 }
 
 
-mainloop()				/* get and process input */
+void
+mainloop(void)				/* get and process input */
 {
 	static struct node	*sis[MAXDEPTH];
 	register struct node	*newp;
@@ -133,19 +145,25 @@ mainloop()				/* get and process input */
 }
 
 
-freetree(tp)				/* free a trace tree */
-struct node	*tp;
+static void
+freetree(				/* free a trace tree */
+	struct node	*tp
+)
 {
-	register struct node	*kid;
+	register struct node	*kid, *k2;
 
-	for (kid = tp->daughter; kid != NULL; kid = kid->sister)
+	for (kid = tp->daughter; kid != NULL; kid = k2) {
+		k2 = kid->sister;
 		freetree(kid);
+	}
 	free((void *)tp);
 }
 
 
-tracerays(tp)				/* trace a ray tree */
-struct node	*tp;
+static void
+tracerays(				/* trace a ray tree */
+	struct node	*tp
+)
 {
 	register struct node	*kid;
 
@@ -156,9 +174,11 @@ struct node	*tp;
 }
 
 
-strtoipt(ipt, str)		/* convert string x y z to image point */
-int	ipt[2];
-char	*str;
+static int
+strtoipt(		/* convert string x y z to image point */
+	int	ipt[2],
+	char	*str
+)
 {
 	FVECT	im_pt, pt;
 
@@ -182,8 +202,10 @@ Window	gwind = 0;
 int	xoff, yoff;
 
 
-setvec(ipt)			/* set up vector drawing for pick */
-int	ipt[2];
+static void
+setvec(			/* set up vector drawing for pick */
+	int	ipt[2]
+)
 {
 	extern Window	xfindwind();
 	XWindowAttributes	wa;
@@ -222,8 +244,11 @@ int	ipt[2];
 }
 
 
-vector(ip1, ip2)		/* draw a vector */
-int	ip1[2], ip2[2];
+static void
+vector(		/* draw a vector */
+	int	ip1[2],
+	int	ip2[2]
+)
 {
 	if (ip2[0] == -1 && ip2[1] == -1)
 		return;			/* null vector */