From 63dde9d4ce1a03007b183d75a396e22a8e9d34ca Mon Sep 17 00:00:00 2001
From: Glenn L McGrath <bug1@ihug.co.nz>
Date: Mon, 18 Sep 2000 09:37:40 +0000
Subject: Simplify and remove dependence on PATH_MAX

buf[PATH_MAX] now defined using strlen to the exact size to be used.
removed output that was displayed if which didnt find the command, this makes itmatch the behaviour of GNU's which.
These modifications result in a slight size decrease.
---
 findutils/which.c | 43 ++++++++++++-------------------------------
 which.c           | 43 ++++++++++++-------------------------------
 2 files changed, 24 insertions(+), 62 deletions(-)

diff --git a/findutils/which.c b/findutils/which.c
index 07c0e0d85..dc162dca5 100644
--- a/findutils/which.c
+++ b/findutils/which.c
@@ -23,14 +23,12 @@
 
 #include "internal.h"
 #include <stdio.h>
-#include <sys/param.h>
 
 extern int which_main(int argc, char **argv)
 {
-	char *path_list, *test, *tmp, *path_parsed;
-	char buf[PATH_MAX];
+	char *path_list, *path_n;
 	struct stat filestat;
-	int count = 0;
+	int i, count=0;
 
 	if (argc <= 1 || **(argv + 1) == '-')
 		usage(which_usage);
@@ -40,45 +38,28 @@ extern int which_main(int argc, char **argv)
 	if (!path_list)
 		path_list = "/bin:/sbin:/usr/bin:/usr/sbin:/usr/local/bin";
 
-	path_parsed = xmalloc (strlen(path_list) + 1);
-	strcpy (path_parsed, path_list);
-
 	/* Replace colons with zeros in path_parsed and count them */
-	count = 1;
-	test = path_parsed;
-	while (1) {
-		tmp = strchr(test, ':');
-		if (tmp == NULL)
-			break;
-		*tmp = 0;
-		test = tmp + 1;
-		count++;
-	}
-
+	for(i=strlen(path_list); i > 0; i--) 
+		if (path_list[i]==':') {
+			path_list[i]=0;
+			count++;
+		}
 
 	while(argc-- > 0) { 
-		int i;
-		int found = FALSE;
-		test = path_parsed;
+		path_n = path_list;
 		argv++;
 		for (i = 0; i < count; i++) {
-			strcpy (buf, test);
+			char buf[strlen(path_n)+1+strlen(*argv)];
+			strcpy (buf, path_n);
 			strcat (buf, "/");
 			strcat (buf, *argv);
 			if (stat (buf, &filestat) == 0
 			    && filestat.st_mode & S_IXUSR)
 			{
-				found = TRUE;
+				printf ("%s\n", buf);
 				break;
 			}
-			test += (strlen(test) + 1);
-		}
-		if (found == TRUE)
-			printf ("%s\n", buf);
-		else
-		{
-			printf ("which: no %s in (%s)\n", *argv, path_list);
-			exit (FALSE);
+			path_n += (strlen(path_n) + 1);
 		}
 	}
 	return(TRUE);
diff --git a/which.c b/which.c
index 07c0e0d85..dc162dca5 100644
--- a/which.c
+++ b/which.c
@@ -23,14 +23,12 @@
 
 #include "internal.h"
 #include <stdio.h>
-#include <sys/param.h>
 
 extern int which_main(int argc, char **argv)
 {
-	char *path_list, *test, *tmp, *path_parsed;
-	char buf[PATH_MAX];
+	char *path_list, *path_n;
 	struct stat filestat;
-	int count = 0;
+	int i, count=0;
 
 	if (argc <= 1 || **(argv + 1) == '-')
 		usage(which_usage);
@@ -40,45 +38,28 @@ extern int which_main(int argc, char **argv)
 	if (!path_list)
 		path_list = "/bin:/sbin:/usr/bin:/usr/sbin:/usr/local/bin";
 
-	path_parsed = xmalloc (strlen(path_list) + 1);
-	strcpy (path_parsed, path_list);
-
 	/* Replace colons with zeros in path_parsed and count them */
-	count = 1;
-	test = path_parsed;
-	while (1) {
-		tmp = strchr(test, ':');
-		if (tmp == NULL)
-			break;
-		*tmp = 0;
-		test = tmp + 1;
-		count++;
-	}
-
+	for(i=strlen(path_list); i > 0; i--) 
+		if (path_list[i]==':') {
+			path_list[i]=0;
+			count++;
+		}
 
 	while(argc-- > 0) { 
-		int i;
-		int found = FALSE;
-		test = path_parsed;
+		path_n = path_list;
 		argv++;
 		for (i = 0; i < count; i++) {
-			strcpy (buf, test);
+			char buf[strlen(path_n)+1+strlen(*argv)];
+			strcpy (buf, path_n);
 			strcat (buf, "/");
 			strcat (buf, *argv);
 			if (stat (buf, &filestat) == 0
 			    && filestat.st_mode & S_IXUSR)
 			{
-				found = TRUE;
+				printf ("%s\n", buf);
 				break;
 			}
-			test += (strlen(test) + 1);
-		}
-		if (found == TRUE)
-			printf ("%s\n", buf);
-		else
-		{
-			printf ("which: no %s in (%s)\n", *argv, path_list);
-			exit (FALSE);
+			path_n += (strlen(path_n) + 1);
 		}
 	}
 	return(TRUE);
-- 
cgit v1.2.3