diff options
author | Glenn L McGrath <bug1@ihug.co.nz> | 2004-02-21 10:49:54 +0300 |
---|---|---|
committer | Glenn L McGrath <bug1@ihug.co.nz> | 2004-02-21 10:49:54 +0300 |
commit | 96099d51b6a872fbd33cdaef123bf93fbc4e9871 (patch) | |
tree | 10e414f256d90f899309d851bd2eacea069ea236 /coreutils | |
parent | ced9705c42d720ce49b99e3c7738969e5fdf63bf (diff) |
Use return instead of exit, use == instead of & ==, left justify labels,
adjustment of whitespace.
Diffstat (limited to 'coreutils')
-rw-r--r-- | coreutils/mv.c | 62 |
1 files changed, 29 insertions, 33 deletions
diff --git a/coreutils/mv.c b/coreutils/mv.c index 55da2cc68..c28d0500e 100644 --- a/coreutils/mv.c +++ b/coreutils/mv.c @@ -41,7 +41,6 @@ static const struct option mv_long_options[] = { { 0, 0, 0, 0 } }; -static const char mv_getopt_short_option[] = "fi"; #define OPT_FILEUTILS_FORCE 1 #define OPT_FILEUTILS_INTERACTIVE 2 @@ -49,21 +48,19 @@ static const char fmt[] = "cannot overwrite %sdirectory with %sdirectory"; extern int mv_main(int argc, char **argv) { - struct stat source_stat; struct stat dest_stat; const char *last; const char *dest; - int dest_exists; - int source_exists; unsigned long flags; + int dest_exists; int status = 0; bb_applet_long_options = mv_long_options; bb_opt_complementaly = "f-i:i-f"; - flags = bb_getopt_ulflags(argc, argv, mv_getopt_short_option); - - if (optind + 2 > argc) + flags = bb_getopt_ulflags(argc, argv, "fi"); + if (optind + 2 > argc) { bb_show_usage(); + } last = argv[argc - 1]; argv += optind; @@ -86,30 +83,34 @@ extern int mv_main(int argc, char **argv) goto RET_1; } - DO_MOVE: +DO_MOVE: if (dest_exists && !(flags & OPT_FILEUTILS_FORCE) && ((access(dest, W_OK) < 0 && isatty(0)) || - (flags & OPT_FILEUTILS_INTERACTIVE))) { - if (fprintf(stderr, "mv: overwrite `%s'? ", dest) < 0) { - goto RET_1; /* Ouch! fprintf failed! */ - } - if (!bb_ask_confirmation()) - goto RET_0; - } - + (flags & OPT_FILEUTILS_INTERACTIVE))) { + if (fprintf(stderr, "mv: overwrite `%s'? ", dest) < 0) { + goto RET_1; /* Ouch! fprintf failed! */ + } + if (!bb_ask_confirmation()) { + goto RET_0; + } + } if (rename(*argv, dest) < 0) { + struct stat source_stat; + int source_exists; + if (errno != EXDEV) { bb_perror_msg("unable to rename `%s'", *argv); - } else if ((source_exists = cp_mv_stat(*argv, &source_stat)) >= 0) { + } + else if ((source_exists = cp_mv_stat(*argv, &source_stat)) >= 0) { if (dest_exists) { - if (dest_exists & 2) { - if (!(source_exists & 2)) { + if (dest_exists == 3) { + if (source_exists != 3) { bb_error_msg(fmt, "", "non-"); goto RET_1; } } else { - if (source_exists & 2) { + if (source_exists == 3) { bb_error_msg(fmt, "non-", ""); goto RET_1; } @@ -118,26 +119,21 @@ extern int mv_main(int argc, char **argv) bb_perror_msg("cannot remove `%s'", dest); goto RET_1; } - } - + } if ((copy_file(*argv, dest, - FILEUTILS_RECUR | FILEUTILS_PRESERVE_STATUS) >= 0) - && (remove_file(*argv, FILEUTILS_RECUR | FILEUTILS_FORCE) >= 0) - ) { + FILEUTILS_RECUR | FILEUTILS_PRESERVE_STATUS) >= 0) && + (remove_file(*argv, FILEUTILS_RECUR | FILEUTILS_FORCE) >= 0)) { goto RET_0; } - } - RET_1: +RET_1: status = 1; } - - RET_0: +RET_0: if (dest != last) { free((void *) dest); - } - + } } while (*++argv != last); - - exit(status); + + return (status); } |