summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorPaul Eggert <eggert@cs.ucla.edu>2004-07-30 04:06:41 +0000
committerPaul Eggert <eggert@cs.ucla.edu>2004-07-30 04:06:41 +0000
commitf232dfa0812d33026213eedca82736cfc69471dd (patch)
tree6488603b2283b3e5549d02a560413f87211575b4 /src
parent6d02317a83678aca779d643e6645bd7ddafe1d5b (diff)
downloadcoreutils-f232dfa0812d33026213eedca82736cfc69471dd.tar.xz
(recurse, force_silent, process_file, process_files, main): Use bool
when appropriate.
Diffstat (limited to 'src')
-rw-r--r--src/chmod.c65
1 files changed, 32 insertions, 33 deletions
diff --git a/src/chmod.c b/src/chmod.c
index 7dea0a24b..36943276b 100644
--- a/src/chmod.c
+++ b/src/chmod.c
@@ -60,11 +60,11 @@ enum Verbosity
/* The name the program was run with. */
char *program_name;
-/* If nonzero, change the modes of directories recursively. */
-static int recurse;
+/* If true, change the modes of directories recursively. */
+static bool recurse;
-/* If nonzero, force silence (no error messages). */
-static int force_silent;
+/* If true, force silence (no error messages). */
+static bool force_silent;
/* Level of verbosity. */
static enum Verbosity verbosity = V_off;
@@ -161,55 +161,55 @@ describe_change (const char *file, mode_t mode,
abort ();
}
printf (fmt, quote (file),
- (unsigned long) (mode & CHMOD_MODE_BITS), &perms[1]);
+ (unsigned long int) (mode & CHMOD_MODE_BITS), &perms[1]);
}
/* Change the mode of FILE according to the list of operations CHANGES.
- Return 0 if successful, -1 if errors occurred. This function is called
+ Return true if successful. This function is called
once for every file system object that fts encounters. */
-static int
+static bool
process_file (FTS *fts, FTSENT *ent, const struct mode_change *changes)
{
char const *file_full_name = ent->fts_path;
char const *file = ent->fts_accpath;
const struct stat *file_stats = ent->fts_statp;
mode_t new_mode IF_LINT (= 0);
- int errors = 0;
+ bool ok = true;
bool do_chmod;
bool symlink_changed = true;
switch (ent->fts_info)
{
case FTS_DP:
- return 0;
+ return true;
case FTS_NS:
error (0, ent->fts_errno, _("cannot access %s"), quote (file_full_name));
- errors = -1;
+ ok = false;
break;
case FTS_ERR:
error (0, ent->fts_errno, _("%s"), quote (file_full_name));
- errors = -1;
+ ok = false;
break;
case FTS_DNR:
error (0, ent->fts_errno, _("cannot read directory %s"),
quote (file_full_name));
- errors = -1;
+ ok = false;
break;
default:
break;
}
- do_chmod = !errors;
+ do_chmod = ok;
if (do_chmod && ROOT_DEV_INO_CHECK (root_dev_ino, file_stats))
{
ROOT_DEV_INO_WARN (file_full_name);
- errors = -1;
+ ok = false;
do_chmod = false;
}
@@ -221,9 +221,9 @@ process_file (FTS *fts, FTSENT *ent, const struct mode_change *changes)
symlink_changed = false;
else
{
- errors = chmod (file, new_mode);
+ ok = (chmod (file, new_mode) == 0);
- if (errors && !force_silent)
+ if (! (ok | force_silent))
error (0, errno, _("changing permissions of %s"),
quote (file_full_name));
}
@@ -232,13 +232,13 @@ process_file (FTS *fts, FTSENT *ent, const struct mode_change *changes)
if (verbosity != V_off)
{
bool changed =
- (!errors && symlink_changed
+ (ok && symlink_changed
&& mode_changed (file, file_stats->st_mode, new_mode));
if (changed || verbosity == V_high)
{
enum Change_status ch_status =
- (errors ? CH_FAILED
+ (!ok ? CH_FAILED
: !symlink_changed ? CH_NOT_APPLIED
: !changed ? CH_NO_CHANGE_REQUESTED
: CH_SUCCEEDED);
@@ -249,19 +249,18 @@ process_file (FTS *fts, FTSENT *ent, const struct mode_change *changes)
if ( ! recurse)
fts_set (fts, ent, FTS_SKIP);
- return errors;
+ return ok;
}
/* Recursively change the modes of the specified FILES (the last entry
of which is NULL) according to the list of operations CHANGES.
BIT_FLAGS controls how fts works.
- If the fts_open call fails, exit nonzero.
- Otherwise, return nonzero upon error. */
+ Return true if successful. */
-static int
+static bool
process_files (char **files, int bit_flags, const struct mode_change *changes)
{
- int fail = 0;
+ bool ok = true;
FTS *fts = xfts_open (files, bit_flags, NULL);
@@ -276,12 +275,12 @@ process_files (char **files, int bit_flags, const struct mode_change *changes)
{
/* FIXME: try to give a better message */
error (0, errno, _("fts_read failed"));
- fail = 1;
+ ok = false;
}
break;
}
- fail |= process_file (fts, ent, changes);
+ ok &= process_file (fts, ent, changes);
}
/* Ignore failure, since the only way it can do so is in failing to
@@ -289,7 +288,7 @@ process_files (char **files, int bit_flags, const struct mode_change *changes)
that doesn't matter. */
fts_close (fts);
- return fail;
+ return ok;
}
void
@@ -340,7 +339,7 @@ int
main (int argc, char **argv)
{
struct mode_change *changes;
- int fail = 0;
+ bool ok;
int modeind = 0; /* Index of the mode argument in `argv'. */
int thisind;
bool preserve_root = false;
@@ -354,7 +353,7 @@ main (int argc, char **argv)
atexit (close_stdout);
- recurse = force_silent = 0;
+ recurse = force_silent = false;
while (1)
{
@@ -402,13 +401,13 @@ main (int argc, char **argv)
reference_file = optarg;
break;
case 'R':
- recurse = 1;
+ recurse = true;
break;
case 'c':
verbosity = V_changes_only;
break;
case 'f':
- force_silent = 1;
+ force_silent = true;
break;
case 'v':
verbosity = V_high;
@@ -444,7 +443,7 @@ main (int argc, char **argv)
error (EXIT_FAILURE, errno, _("failed to get attributes of %s"),
quote (reference_file));
- if (recurse && preserve_root)
+ if (recurse & preserve_root)
{
static struct dev_ino dev_ino_buf;
root_dev_ino = get_root_dev_ino (&dev_ino_buf);
@@ -457,7 +456,7 @@ main (int argc, char **argv)
root_dev_ino = NULL;
}
- fail = process_files (argv + optind, FTS_COMFOLLOW, changes);
+ ok = process_files (argv + optind, FTS_COMFOLLOW, changes);
- exit (fail ? EXIT_FAILURE : EXIT_SUCCESS);
+ exit (ok ? EXIT_SUCCESS : EXIT_FAILURE);
}