svn commit: trunk/busybox/archival/libunarchive

vda at busybox.net vda at busybox.net
Wed Mar 14 22:08:54 UTC 2007


Author: vda
Date: 2007-03-14 15:08:53 -0700 (Wed, 14 Mar 2007)
New Revision: 18112

Log:
get_header_ar: reformatted code, no real changes


Modified:
   trunk/busybox/archival/libunarchive/get_header_ar.c


Changeset:
Modified: trunk/busybox/archival/libunarchive/get_header_ar.c
===================================================================
--- trunk/busybox/archival/libunarchive/get_header_ar.c	2007-03-14 22:08:04 UTC (rev 18111)
+++ trunk/busybox/archival/libunarchive/get_header_ar.c	2007-03-14 22:08:53 UTC (rev 18112)
@@ -66,7 +66,9 @@
 
 	/* long filenames have '/' as the first character */
 	if (ar.formatted.name[0] == '/') {
-#ifdef CONFIG_FEATURE_AR_LONG_FILENAMES
+		unsigned long_offset;
+
+#if ENABLE_FEATURE_AR_LONG_FILENAMES
 		if (ar.formatted.name[1] == '/') {
 			/* If the second char is a '/' then this entries data section
 			 * stores long filename for multiple entries, they are stored
@@ -78,20 +80,22 @@
 			/* This ar entries data section only contained filenames for other records
 			 * they are stored in the static ar_long_names for future reference */
 			return get_header_ar(archive_handle); /* Return next header */
-		} else if (ar.formatted.name[1] == ' ') {
+		}
+
+		if (ar.formatted.name[1] == ' ') {
 			/* This is the index of symbols in the file for compilers */
 			data_skip(archive_handle);
 			archive_handle->offset += typed->size;
 			return get_header_ar(archive_handle); /* Return next header */
-		} else {
-			/* The number after the '/' indicates the offset in the ar data section
-			(saved in variable long_name) that conatains the real filename */
-			const unsigned int long_offset = atoi(&ar.formatted.name[1]);
-			if (long_offset >= ar_long_name_size) {
-				bb_error_msg_and_die("can't resolve long filename");
-			}
-			typed->name = xstrdup(ar_long_names + long_offset);
 		}
+
+		/* The number after the '/' indicates the offset in the ar data section
+		 * (saved in variable long_name) that conatains the real filename */
+		long_offset = atoi(&ar.formatted.name[1]);
+		if (long_offset >= ar_long_name_size) {
+			bb_error_msg_and_die("can't resolve long filename");
+		}
+		typed->name = xstrdup(ar_long_names + long_offset);
 #else
 		bb_error_msg_and_die("long filenames not supported");
 #endif




More information about the busybox-cvs mailing list