Run through indent, use braces
diff --git a/archival/gunzip.c b/archival/gunzip.c
index 4ab197f..e379ccf 100644
--- a/archival/gunzip.c
+++ b/archival/gunzip.c
@@ -71,90 +71,95 @@
const int gunzip_test = 4;
const int gunzip_verbose = 8;
-static int gunzip_file (const char *path, int flags)
+static int gunzip_file(const char *path, int flags)
{
- FILE *in_file, *out_file;
- struct stat stat_buf;
- const char *delete_path = NULL;
- char *out_path = NULL;
+ FILE *in_file, *out_file;
+ struct stat stat_buf;
+ const char *delete_path = NULL;
+ char *out_path = NULL;
- if (path == NULL || strcmp (path, "-") == 0) {
- in_file = stdin;
- flags |= gunzip_to_stdout;
- } else {
- if ((in_file = wfopen(path, "r")) == NULL)
- return -1;
-
- if (flags & gunzip_verbose) {
- fprintf(stderr, "%s:\t", path);
- }
-
- /* set the buffer size */
- setvbuf(in_file, NULL, _IOFBF, 0x8000);
-
- /* Get the time stamp on the input file. */
- if (stat(path, &stat_buf) < 0) {
- error_msg_and_die("Couldn't stat file %s", path);
- }
- }
-
- /* Check that the input is sane. */
- if (isatty(fileno(in_file)) && (flags & gunzip_force) == 0)
- error_msg_and_die("compressed data not read from terminal. Use -f to force it.");
-
- /* Set output filename and number */
- if (flags & gunzip_test) {
- out_file = xfopen("/dev/null", "w"); /* why does test use filenum 2 ? */
- } else if (flags & gunzip_to_stdout) {
- out_file = stdout;
- } else {
- char *extension;
- int length = strlen(path);
-
- extension = strrchr(path, '.');
- if (extension && strcmp(extension, ".gz") == 0) {
- length -= 3;
- } else if (extension && strcmp(extension, ".tgz") == 0) {
- length -= 4;
+ if (path == NULL || strcmp(path, "-") == 0) {
+ in_file = stdin;
+ flags |= gunzip_to_stdout;
} else {
- error_msg_and_die("Invalid extension");
+ if ((in_file = wfopen(path, "r")) == NULL) {
+ return -1;
+ }
+ if (flags & gunzip_verbose) {
+ fprintf(stderr, "%s:\t", path);
+ }
+
+ /* set the buffer size */
+ setvbuf(in_file, NULL, _IOFBF, 0x8000);
+
+ /* Get the time stamp on the input file. */
+ if (stat(path, &stat_buf) < 0) {
+ error_msg_and_die("Couldn't stat file %s", path);
+ }
}
- out_path = xstrndup(path, length);
-
- /* Open output file */
- out_file = xfopen(out_path, "w");
- /* Set permissions on the file */
- chmod(out_path, stat_buf.st_mode);
- }
-
- /* do the decompression, and cleanup */
- if (unzip(in_file, out_file) == 0) {
- /* Success, remove .gz file */
- if ( !(flags & gunzip_to_stdout ))
- delete_path = path;
- if (flags & gunzip_verbose) {
- fprintf(stderr, "OK\n");
+ /* Check that the input is sane. */
+ if (isatty(fileno(in_file)) && (flags & gunzip_force) == 0) {
+ error_msg_and_die
+ ("compressed data not read from terminal. Use -f to force it.");
}
- } else {
- /* remove failed attempt */
- delete_path = out_path;
- }
- if (out_file != stdout)
- fclose(out_file);
- if (in_file != stdin)
- fclose(in_file);
+ /* Set output filename and number */
+ if (flags & gunzip_test) {
+ out_file = xfopen("/dev/null", "w"); /* why does test use filenum 2 ? */
+ } else if (flags & gunzip_to_stdout) {
+ out_file = stdout;
+ } else {
+ char *extension;
+ int length = strlen(path);
- if (delete_path && !(flags & gunzip_test)) {
- if (unlink(delete_path) < 0) {
- error_msg_and_die("Couldn't remove %s", delete_path);
+ extension = strrchr(path, '.');
+ if (extension && strcmp(extension, ".gz") == 0) {
+ length -= 3;
+ } else if (extension && strcmp(extension, ".tgz") == 0) {
+ length -= 4;
+ } else {
+ error_msg_and_die("Invalid extension");
+ }
+ out_path = xstrndup(path, length);
+
+ /* Open output file */
+ out_file = xfopen(out_path, "w");
+
+ /* Set permissions on the file */
+ chmod(out_path, stat_buf.st_mode);
}
- }
- free(out_path);
+ /* do the decompression, and cleanup */
+ if (unzip(in_file, out_file) == 0) {
+ /* Success, remove .gz file */
+ if (!(flags & gunzip_to_stdout)) {
+ delete_path = path;
+ }
+ if (flags & gunzip_verbose) {
+ fprintf(stderr, "OK\n");
+ }
+ } else {
+ /* remove failed attempt */
+ delete_path = out_path;
+ }
- return 0;
+ if (out_file != stdout) {
+ fclose(out_file);
+ }
+ if (in_file != stdin) {
+ fclose(in_file);
+ }
+
+ if (delete_path && !(flags & gunzip_test)) {
+ if (unlink(delete_path) < 0) {
+ error_msg_and_die("Couldn't remove %s", delete_path);
+ }
+ }
+
+ free(out_path);
+
+ return 0;
}
extern int gunzip_main(int argc, char **argv)
@@ -164,8 +169,9 @@
int status = EXIT_SUCCESS;
/* if called as zcat */
- if (strcmp(applet_name, "zcat") == 0)
+ if (strcmp(applet_name, "zcat") == 0) {
flags |= gunzip_to_stdout;
+ }
while ((opt = getopt(argc, argv, "ctfhdqv")) != -1) {
switch (opt) {
@@ -181,25 +187,27 @@
case 'v':
flags |= gunzip_verbose;
break;
- case 'd': /* Used to convert gzip to gunzip. */
+ case 'd': /* Used to convert gzip to gunzip. */
break;
case 'q':
error_msg("-q option not supported, ignored");
break;
case 'h':
default:
- show_usage(); /* exit's inside usage */
+ show_usage(); /* exit's inside usage */
}
}
if (optind == argc) {
- if (gunzip_file (NULL, flags) < 0)
- status = EXIT_FAILURE;
+ if (gunzip_file(NULL, flags) < 0) {
+ status = EXIT_FAILURE;
+ }
} else {
- for (i = optind; i < argc; i++) {
- if (gunzip_file (argv[i], flags) < 0)
- status = EXIT_FAILURE;
- }
+ for (i = optind; i < argc; i++) {
+ if (gunzip_file(argv[i], flags) < 0) {
+ status = EXIT_FAILURE;
+ }
+ }
}
return status;
}