Don't comment warnings, _FIX_ warnings. (And putting in #warnings about

other warnings is just gross.)

On a side note, while I was there, I made the code slightly smaller.

1 files changed, 22 insertions, 48 deletions

diff --git a/archival/bunzip2.c b/archival/bunzip2.c
index 5cd013c0c..740e26919 100644
--- a/archival/bunzip2.c
+++ b/archival/bunzip2.c
@@ -2,19 +2,7 @@
  *  Modified for busybox by Glenn McGrath <bug1@iinet.net.au>
  *  Added support output to stdout by Thomas Lundquist <thomasez@zelow.no>
  *
- *  This program is free software; you can redistribute it and/or modify
- *  it under the terms of the GNU General Public License as published by
- *  the Free Software Foundation; either version 2 of the License, or
- *  (at your option) any later version.
- *
- *  This program is distributed in the hope that it will be useful,
- *  but WITHOUT ANY WARRANTY; without even the implied warranty of
- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *  GNU General Public License for more details.
- *
- *  You should have received a copy of the GNU General Public License
- *  along with this program; if not, write to the Free Software
- *  Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+ *  Licensed under GPL v2, see file LICENSE in this tarball for details.
  */
 
 #include <fcntl.h>
@@ -32,61 +20,47 @@
 
 int bunzip2_main(int argc, char **argv)
 {
-	char *compressed_name;
-	/* Note: Ignore the warning about save_name being used uninitialized.
-	 * That is not the case, but gcc has trouble working that out... */
-#warning The save_name warning is OK, ignore it
-	char *save_name;
+	char *filename;
 	unsigned long opt;
-	int status;
-	int src_fd;
-	int dst_fd;
+	int status, src_fd, dst_fd;
 
 	opt = bb_getopt_ulflags(argc, argv, "cf");
 
-	/* if called as bzcat force the stdout flag */
-	if (bb_applet_name[2] == 'c') {
-		opt |= BUNZIP2_OPT_STDOUT;
-	}
-
 	/* Set input filename and number */
-	compressed_name = argv[optind];
-	if ((compressed_name) && (compressed_name[0] != '-') && (compressed_name[1] != '\0')) {
+	filename = argv[optind];
+	if ((filename) && (filename[0] != '-') && (filename[1] != '\0')) {
 		/* Open input file */
-		src_fd = bb_xopen(compressed_name, O_RDONLY);
+		src_fd = bb_xopen(filename, O_RDONLY);
 	} else {
 		src_fd = STDIN_FILENO;
-		opt |= BUNZIP2_OPT_STDOUT;
+		filename = 0;
 	}
+	
+	/* if called as bzcat force the stdout flag */
+	if ((opt & BUNZIP2_OPT_STDOUT) || bb_applet_name[2] == 'c')
+		filename = 0;
 
 	/* Check that the input is sane.  */
 	if (isatty(src_fd) && (opt & BUNZIP2_OPT_FORCE) == 0) {
 		bb_error_msg_and_die("compressed data not read from terminal.  Use -f to force it.");
 	}
 
-	if (opt & BUNZIP2_OPT_STDOUT) {
-		dst_fd = STDOUT_FILENO;
-	} else {
-		int len = strlen(compressed_name) - 4;
-		if (strcmp(compressed_name + len, ".bz2") != 0) {
+	if (filename) {
+		char *extension=filename+strlen(filename)-4;
+		if (strcmp(extension, ".bz2") != 0) {
 			bb_error_msg_and_die("Invalid extension");
 		}
-		save_name = bb_xstrndup(compressed_name, len);
-		dst_fd = bb_xopen(save_name, O_WRONLY | O_CREAT);
-	}
-
+		*extension=0;
+		dst_fd = bb_xopen(filename, O_WRONLY | O_CREAT);
+	} else dst_fd = STDOUT_FILENO;
 	status = uncompressStream(src_fd, dst_fd);
-	if(!(opt & BUNZIP2_OPT_STDOUT)) {
-		char *delete_name;
-		if (status) {
-			delete_name = save_name;
-		} else {
-			delete_name = compressed_name;
-		}
-		if (unlink(delete_name) < 0) {
-			bb_error_msg_and_die("Couldn't remove %s", delete_name);
+	if(filename) {
+		if (!status) filename[strlen(filename)]='.';
+		if (unlink(filename) < 0) {
+			bb_error_msg_and_die("Couldn't remove %s", filename);
 		}
 	}
 
 	return status;
 }
+/* vi:set ts=4: */