diff options
author | Jim Meyering <meyering@redhat.com> | 2011-12-03 17:49:57 +0100 |
---|---|---|
committer | Jim Meyering <meyering@redhat.com> | 2011-12-04 10:31:03 +0100 |
commit | 63098ee58235f5a1877220e8c073f6f23b09264f (patch) | |
tree | bd2fe30d768fc5cdb499657722e5cdea40664902 | |
parent | bea7b10489afcc845db00b03da6ccea71de6cb1d (diff) | |
download | coreutils-63098ee58235f5a1877220e8c073f6f23b09264f.tar.xz |
od,test: address warnings from gcc's -Wjump-misses-init
* src/test.c (unary_operator): gcc reported that initializations
in two case statements were skipped. Enclose in braces.
* src/od.c (decode_one_format): Likewise.
-rw-r--r-- | src/od.c | 46 | ||||
-rw-r--r-- | src/test.c | 32 |
2 files changed, 42 insertions, 36 deletions
@@ -771,32 +771,34 @@ this system doesn't provide a %lu-byte floating point type"), } size_spec = fp_type_size[size]; - struct lconv const *locale = localeconv (); - size_t decimal_point_len = - (locale->decimal_point[0] ? strlen (locale->decimal_point) : 1); - - switch (size_spec) - { - case FLOAT_SINGLE: - print_function = print_float; - field_width = FLT_STRLEN_BOUND_L (decimal_point_len); - break; + { + struct lconv const *locale = localeconv (); + size_t decimal_point_len = + (locale->decimal_point[0] ? strlen (locale->decimal_point) : 1); + + switch (size_spec) + { + case FLOAT_SINGLE: + print_function = print_float; + field_width = FLT_STRLEN_BOUND_L (decimal_point_len); + break; - case FLOAT_DOUBLE: - print_function = print_double; - field_width = DBL_STRLEN_BOUND_L (decimal_point_len); - break; + case FLOAT_DOUBLE: + print_function = print_double; + field_width = DBL_STRLEN_BOUND_L (decimal_point_len); + break; - case FLOAT_LONG_DOUBLE: - print_function = print_long_double; - field_width = LDBL_STRLEN_BOUND_L (decimal_point_len); - break; + case FLOAT_LONG_DOUBLE: + print_function = print_long_double; + field_width = LDBL_STRLEN_BOUND_L (decimal_point_len); + break; - default: - abort (); - } + default: + abort (); + } - break; + break; + } case 'a': ++s; diff --git a/src/test.c b/src/test.c index 1b06ca86e..a5df7c8dc 100644 --- a/src/test.c +++ b/src/test.c @@ -413,22 +413,26 @@ unary_operator (void) return euidaccess (argv[pos - 1], X_OK) == 0; case 'O': /* File is owned by you? */ - unary_advance (); - if (stat (argv[pos - 1], &stat_buf) != 0) - return false; - errno = 0; - uid_t euid = geteuid (); - uid_t NO_UID = -1; - return ! (euid == NO_UID && errno) && euid == stat_buf.st_uid; + { + unary_advance (); + if (stat (argv[pos - 1], &stat_buf) != 0) + return false; + errno = 0; + uid_t euid = geteuid (); + uid_t NO_UID = -1; + return ! (euid == NO_UID && errno) && euid == stat_buf.st_uid; + } case 'G': /* File is owned by your group? */ - unary_advance (); - if (stat (argv[pos - 1], &stat_buf) != 0) - return false; - errno = 0; - gid_t egid = getegid (); - gid_t NO_GID = -1; - return ! (egid == NO_GID && errno) && egid == stat_buf.st_gid; + { + unary_advance (); + if (stat (argv[pos - 1], &stat_buf) != 0) + return false; + errno = 0; + gid_t egid = getegid (); + gid_t NO_GID = -1; + return ! (egid == NO_GID && errno) && egid == stat_buf.st_gid; + } case 'f': /* File is a file? */ unary_advance (); |