svn commit: trunk/busybox: libbb procps

[aldot at busybox.net]

Author: aldot
Date: 2005-10-07 08:44:37 -0700 (Fri, 07 Oct 2005)
New Revision: 11799

Log:
- size reduction of pidof
- fix typo in reverse_pidlist


Modified:
   trunk/busybox/libbb/find_pid_by_name.c
   trunk/busybox/procps/pidof.c


Changeset:
Modified: trunk/busybox/libbb/find_pid_by_name.c
===================================================================
--- trunk/busybox/libbb/find_pid_by_name.c	2005-10-07 15:36:26 UTC (rev 11798)
+++ trunk/busybox/libbb/find_pid_by_name.c	2005-10-07 15:44:37 UTC (rev 11799)
@@ -45,7 +45,7 @@
 extern long *pidlist_reverse(long *pidList)
 {
 	int i=0;
-	while (pidList[i] > 0 && i++);
+	while (pidList[i] > 0 && ++i);
 	if ( i-- > 0) {
 		long k;
 		int j;

Modified: trunk/busybox/procps/pidof.c
===================================================================
--- trunk/busybox/procps/pidof.c	2005-10-07 15:36:26 UTC (rev 11798)
+++ trunk/busybox/procps/pidof.c	2005-10-07 15:44:37 UTC (rev 11799)
@@ -15,6 +15,7 @@
 #include <signal.h>
 #include <ctype.h>
 #include <string.h>
+#include <sys/types.h>
 #include <unistd.h>
 #include "busybox.h"
 
@@ -29,7 +30,6 @@
 #if ENABLE_FEATURE_PIDOF_OMIT
 #define _OMIT_COMPL(a) a
 #define _OMIT(a) ,a
-static llist_t *omits; /* list of pids to omit */
 #if ENABLE_FEATURE_PIDOF_SINGLE
 #define OMIT (1<<1)
 #else
@@ -39,59 +39,46 @@
 #define _OMIT_COMPL(a) ""
 #define _OMIT(a)
 #define OMIT (0)
+#define omitted (0)
 #endif
 
 extern int pidof_main(int argc, char **argv)
 {
-	int n = 0;
-	int fail = 1;
+	unsigned n = 0;
+	unsigned fail = 1;
 	unsigned long int opt;
 #if ENABLE_FEATURE_PIDOF_OMIT
+	llist_t *omits = NULL; /* list of pids to omit */
 	bb_opt_complementally = _OMIT_COMPL("o*");
 #endif
 
-	/* do option parsing */
-	if ((opt = bb_getopt_ulflags(argc, argv,
+	/* do unconditional option parsing */
+	opt = bb_getopt_ulflags(argc, argv,
 					_SINGLE_COMPL("s") _OMIT_COMPL("o:")
-					_OMIT(&omits)))
-			> 0) {
-#if ENABLE_FEATURE_PIDOF_SINGLE
-		if (!(opt & SINGLE))
-#endif
-#if ENABLE_FEATURE_PIDOF_OMIT
-		if (!(opt & OMIT))
-#endif
-		bb_show_usage();
-	}
+					_OMIT(&omits));
 
 #if ENABLE_FEATURE_PIDOF_OMIT
 	/* fill omit list.  */
 	{
-	RESERVE_CONFIG_BUFFER(getppid_str, 32);
-	llist_t * omits_p = omits;
-	while (omits_p) {
-		/* are we asked to exclude the parent's process ID?  */
-		if (omits_p->data[0] == '%') {
+		RESERVE_CONFIG_BUFFER(getppid_str, 32);
+		llist_t * omits_p = omits;
+		while (omits_p) {
+			/* are we asked to exclude the parent's process ID?  */
 			if (!strncmp(omits_p->data, "%PPID", 5)) {
-				/* yes, exclude ppid */
-				snprintf(getppid_str, sizeof(getppid_str), "%ld", getppid());
-//				omits_p->data = getppid_str;
+				omits_p = llist_free_one(omits_p);
+				snprintf(getppid_str, sizeof(getppid_str), "%d", getppid());
+				omits_p = llist_add_to(omits_p, getppid_str);
 #if 0
 			} else {
 				bb_error_msg_and_die("illegal omit pid value (%s)!\n",
 						omits_p->data);
 #endif
 			}
-		} else {
-		/* no, not talking about ppid but deal with usual case (pid).  */
-			snprintf(getppid_str, sizeof(getppid_str), "%ld",
-					strtol(omits_p->data, NULL, 10));
+			omits_p = omits_p->link;
 		}
-		omits_p->data = getppid_str;
-		omits_p = omits_p->link;
+		if (ENABLE_FEATURE_CLEAN_UP)
+			RELEASE_CONFIG_BUFFER(getppid_str);
 	}
-	RELEASE_CONFIG_BUFFER(getppid_str);
-	}
 #endif
 	/* Looks like everything is set to go.  */
 	while(optind < argc) {
@@ -102,7 +89,7 @@
 		pidList = pidlist_reverse(find_pid_by_name(argv[optind]));
 		for(pl = pidList; *pl > 0; pl++) {
 #if ENABLE_FEATURE_PIDOF_OMIT
-			unsigned short omitted = 0;
+			unsigned omitted = 0;
 			if (opt & OMIT) {
 				llist_t *omits_p = omits;
 				while (omits_p)
@@ -111,18 +98,19 @@
 					} else
 						omits_p = omits_p->link;
 			}
-			if (!omitted)
 #endif
-			printf("%s%ld", (n++ ? " " : ""), *pl);
-#if ENABLE_FEATURE_PIDOF_OMIT
-			fail = omitted;
-#else
-			fail = 0;
-#endif
-#if ENABLE_FEATURE_PIDOF_SINGLE
-			if (opt & SINGLE)
+			if (!omitted) {
+				if (n) {
+					putchar(' ');
+				} else {
+					n = 1;
+				}
+				printf("%ld", *pl);
+			}
+			fail = (!ENABLE_FEATURE_PIDOF_OMIT && omitted);
+
+			if (ENABLE_FEATURE_PIDOF_SINGLE && (opt & SINGLE))
 				break;
-#endif
 		}
 		free(pidList);
 		optind++;