Update applet define from BB_CP_MV to BB_CP and BB_MV.

[oweals/busybox.git] / rm.c

diff --git a/rm.c b/rm.c
index f1152eab37389cba7ad6f79bdb37275c124cce4f..ea7f86ce100bfe7309a4b954e997cb480e936011 100644 (file)
--- a/rm.c
+++ b/rm.c
@@ -33,6 +33,7 @@
 #include <errno.h>
 #include <unistd.h>
 #include <stdlib.h>
+#include <getopt.h>
 #include "busybox.h"
 
 static int recursiveFlag = FALSE;
@@ -82,52 +83,46 @@ static int dirAction(const char *fileName, struct stat *statbuf, void* junk)
 
 extern int rm_main(int argc, char **argv)
 {
+       int opt;
        int status = EXIT_SUCCESS;
-       int stopIt=FALSE;
        struct stat statbuf;
-
-       argc--;
-       argv++;
-
-       /* Parse any options */
-       while (argc > 0 && stopIt == FALSE) {
-               if (**argv == '-') {
-                       while (*++(*argv))
-                               switch (**argv) {
-                                       case 'R':
-                                       case 'r':
-                                               recursiveFlag = TRUE;
-                                               break;
-                                       case 'f':
-                                               forceFlag = TRUE;
+       
+       
+       /* do normal option parsing */
+       while ((opt = getopt(argc, argv, "Rrf-"
+#ifdef BB_FEATURE_RM_INTERACTIVE
+"i"
+#endif
+)) > 0) {
+               switch (opt) {
+                       case 'R':
+                       case 'r':
+                               recursiveFlag = TRUE;
+                               break;
+                       case 'f':
+                               forceFlag = TRUE;
 #ifdef BB_FEATURE_RM_INTERACTIVE
-                                               interactiveFlag = FALSE;
+
+                               interactiveFlag = FALSE;
 #endif
-                                               break;
-                                       case 'i':
+                               break;
 #ifdef BB_FEATURE_RM_INTERACTIVE
-                                               interactiveFlag = TRUE;
+                       case 'i':
+                               interactiveFlag = TRUE;
+                               forceFlag = FALSE;
+                               break;
 #endif
-                                               break;
-                                       case '-':
-                                               stopIt = TRUE;
-                                               break;
-                                       default:
-                                               show_usage();
-                               }
-                       argc--;
-                       argv++;
+                       default:
+                               show_usage();
                }
-               else
-                       break;
        }
-
-       if (argc < 1 && forceFlag == FALSE) {
+       
+       if (argc == optind && forceFlag == FALSE) {
                show_usage();
        }
 
-       while (argc-- > 0) {
-               srcName = *(argv++);
+       while (optind < argc) {
+               srcName = argv[optind];
                if (forceFlag == TRUE && lstat(srcName, &statbuf) != 0
                        && errno == ENOENT) {
                        /* do not reports errors for non-existent files if -f, just skip them */
@@ -137,6 +132,7 @@ extern int rm_main(int argc, char **argv)
                                status = EXIT_FAILURE;
                        }
                }
+               optind++;
        }
        return status;
 }