[BusyBox-cvs] busybox/miscutils crond.c,1.3,1.4

Erik Andersen andersen at busybox.net
Fri Jun 20 09:02:01 UTC 2003


Update of /var/cvs/busybox/miscutils
In directory winder:/tmp/cvs-serv14650/miscutils

Modified Files:
	crond.c 
Log Message:
last_patch89 from vodz:

    Manuel,

    I rewrite bb_getopt_ulflags() function for more universal usage.
    My version support now:
    - options with arguments (optional arg as GNU extension also)
    - complementaly and/or incomplementaly and/or incongruously and/or list
    options
    - long_opt (all applets may have long option, add supporting is trivial)
    This realisation full compatibile from your version.
    Code size grow 480 bytes, but only coreutils/* over compensate this size
    after using new function. Last patch reduced over 800 bytes and not full
    applied to all. "mkdir" and "mv" applets have long_opt now for demonstrate
    trivial addition support long_opt with usage new bb_getopt_ulflags().
    Complementaly and/or incomplementaly and/or incongruously and/or list options
    logic is not trivial, but new "cut" and "grep" applets using this logic
    for examples with full demostrating. New "grep" applet reduced over 300
    bytes.

    Mark,
    Also. I removed bug from "grep" applet.
    $ echo a b | busybox grep -e a b
    a b
    a b
    But right is printing one only.

    --w
    vodz



Index: crond.c
===================================================================
RCS file: /var/cvs/busybox/miscutils/crond.c,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -d -r1.3 -r1.4
--- crond.c	19 Mar 2003 09:12:28 -0000	1.3
+++ crond.c	20 Jun 2003 09:01:58 -0000	1.4
@@ -117,46 +117,40 @@
 int
 crond_main(int ac, char **av)
 {
-    int i;
+    unsigned long opt;
+    char *lopt, *Lopt, *copt;
+#ifdef FEATURE_DEBUG_OPT
+    char *dopt;
+    bb_opt_complementaly = "f-b:b-f:S-L:L-S:d-l";
+#else
+    bb_opt_complementaly = "f-b:b-f:S-L:L-S";
+#endif
 
     opterr = 0;         /* disable getopt 'errors' message.*/
-
-    while ((i = getopt(ac,av,
+    opt = bb_getopt_ulflags(ac, av, "l:L:fbSc:"
 #ifdef FEATURE_DEBUG_OPT
 				"d:"
 #endif
-					"l:L:fbSc:")) != EOF){
-
-	switch (i){
-	    case 'l':
-		LogLevel = atoi(optarg);
-		break;
+	    , &lopt, &Lopt, &copt
 #ifdef FEATURE_DEBUG_OPT
-	    case 'd':
-		DebugOpt = atoi(optarg);
-		LogLevel = 0;
-		break;
+	    , &dopt
 #endif
-	    case 'f':
-		ForegroundOpt = 1;
-		break;
-	    case 'b':
-		ForegroundOpt = 0;
-		break;
-	    case 'S':                   /* select logging to syslog */
-		LoggerOpt = 0;
-		break;
-	    case 'L':                   /* select internal file logger */
-		LoggerOpt = 1;
-		if (*optarg != 0) LogFile = optarg;
-		break;
-	    case 'c':
-		if (*optarg != 0) CDir = optarg;
-		break;
-	    default: /*  parse error */
-		bb_show_usage();
+	    );
+    if(opt & 1)
+	LogLevel = atoi(lopt);
+    LoggerOpt = opt & 2;
+    if(LoggerOpt)
+	if (*Lopt != 0) LogFile = Lopt;
+    ForegroundOpt = opt & 4;
+    if(opt & 32) {
+	if (*copt != 0) CDir = copt;
 	}
+#ifdef FEATURE_DEBUG_OPT
+    if(opt & 64) {
+	DebugOpt = atoi(dopt);
+	LogLevel = 0;
     }
+#endif
 
     /*
      * change directory
@@ -165,6 +159,9 @@
     if (chdir(CDir) != 0)
 	bb_perror_msg_and_die("chdir");
 
+    signal(SIGHUP,SIG_IGN);   /* hmm.. but, if kill -HUP original
+				 * version - his died. ;(
+				 */
     /*
      * close stdin and stdout, stderr.
      * close unused descriptors -  don't need.
@@ -177,9 +174,6 @@
     }
 
     (void)startlogger();                /* need if syslog mode selected */
-    signal(SIGHUP,SIG_IGN);   /* hmm.. but, if kill -HUP original
-				 * version - his died. ;(
-				 */
 
     /*
      * main loop - synchronize to 1 second after the minute, minimum sleep



More information about the busybox-cvs mailing list