svn commit: trunk/busybox: archival coreutils editors findutils inc etc...

vda at busybox.net vda at busybox.net
Thu Oct 12 22:43:21 UTC 2006


Author: vda
Date: 2006-10-12 15:43:20 -0700 (Thu, 12 Oct 2006)
New Revision: 16375

Log:
bb_get_[chomped]line_from_file wasn't descriptive enough.
Renaming...


Modified:
   trunk/busybox/archival/dpkg.c
   trunk/busybox/archival/tar.c
   trunk/busybox/coreutils/cut.c
   trunk/busybox/coreutils/md5_sha1_sum.c
   trunk/busybox/coreutils/sort.c
   trunk/busybox/coreutils/uniq.c
   trunk/busybox/coreutils/uudecode.c
   trunk/busybox/editors/patch.c
   trunk/busybox/editors/sed.c
   trunk/busybox/findutils/grep.c
   trunk/busybox/include/libbb.h
   trunk/busybox/libbb/get_line_from_file.c
   trunk/busybox/miscutils/dc.c
   trunk/busybox/miscutils/makedevs.c
   trunk/busybox/networking/dnsd.c
   trunk/busybox/networking/ifupdown.c
   trunk/busybox/networking/nameif.c
   trunk/busybox/shell/cmdedit.c
   trunk/busybox/util-linux/hexdump.c
   trunk/busybox/util-linux/mount.c


Changeset:
Modified: trunk/busybox/archival/dpkg.c
===================================================================
--- trunk/busybox/archival/dpkg.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/archival/dpkg.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -1206,7 +1206,7 @@
 		return NULL;
 	}
 
-	while ((line = bb_get_chomped_line_from_file(list_stream)) != NULL) {
+	while ((line = xmalloc_getline(list_stream)) != NULL) {
 		file_list = xrealloc(file_list, sizeof(char *) * (count + 2));
 		file_list[count] = line;
 		count++;

Modified: trunk/busybox/archival/tar.c
===================================================================
--- trunk/busybox/archival/tar.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/archival/tar.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -523,7 +523,7 @@
 		tmp = cur;
 		cur = cur->link;
 		free(tmp);
-		while ((line = bb_get_chomped_line_from_file(src_stream)) != NULL) {
+		while ((line = xmalloc_getline(src_stream)) != NULL) {
 			char *filename_ptr = last_char_is(line, '/');
 			if (filename_ptr > line)
 				*filename_ptr = '\0';

Modified: trunk/busybox/coreutils/cut.c
===================================================================
--- trunk/busybox/coreutils/cut.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/coreutils/cut.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -50,7 +50,7 @@
 	unsigned int linenum = 0;	/* keep these zero-based to be consistent */
 
 	/* go through every line in the file */
-	while ((line = bb_get_chomped_line_from_file(file)) != NULL) {
+	while ((line = xmalloc_getline(file)) != NULL) {
 
 		/* set up a list so we can keep track of what's been printed */
 		char * printed = xzalloc(strlen(line) * sizeof(char));

Modified: trunk/busybox/coreutils/md5_sha1_sum.c
===================================================================
--- trunk/busybox/coreutils/md5_sha1_sum.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/coreutils/md5_sha1_sum.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -126,7 +126,7 @@
 			pre_computed_stream = xfopen(file_ptr, "r");
 		}
 
-		while ((line = bb_get_chomped_line_from_file(pre_computed_stream)) != NULL) {
+		while ((line = xmalloc_getline(pre_computed_stream)) != NULL) {
 			char *filename_ptr;
 
 			count_total++;

Modified: trunk/busybox/coreutils/sort.c
===================================================================
--- trunk/busybox/coreutils/sort.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/coreutils/sort.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -124,9 +124,9 @@
 }
 
 #define GET_LINE(fp) (global_flags&FLAG_z) ? bb_get_chunk_from_file(fp,NULL) \
-		: bb_get_chomped_line_from_file(fp)
+		: xmalloc_getline(fp)
 #else
-#define GET_LINE(fp)	bb_get_chomped_line_from_file(fp)
+#define GET_LINE(fp)	xmalloc_getline(fp)
 #endif
 
 /* Iterate through keys list and perform comparisons */

Modified: trunk/busybox/coreutils/uniq.c
===================================================================
--- trunk/busybox/coreutils/uniq.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/coreutils/uniq.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -69,7 +69,7 @@
 		dups = 0;
 
 		/* gnu uniq ignores newlines */
-		while ((s1 = bb_get_chomped_line_from_file(in)) != NULL) {
+		while ((s1 = xmalloc_getline(in)) != NULL) {
 			e1 = s1;
 			for (i=skip_fields ; i ; i--) {
 				e1 = skip_whitespace(e1);

Modified: trunk/busybox/coreutils/uudecode.c
===================================================================
--- trunk/busybox/coreutils/uudecode.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/coreutils/uudecode.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -18,7 +18,7 @@
 {
 	char *line;
 
-	while ((line = bb_get_chomped_line_from_file(src_stream)) != NULL) {
+	while ((line = xmalloc_getline(src_stream)) != NULL) {
 		int length;
 		char *line_ptr = line;
 
@@ -140,7 +140,7 @@
 	}
 
 	/* Search for the start of the encoding */
-	while ((line = bb_get_chomped_line_from_file(src_stream)) != NULL) {
+	while ((line = xmalloc_getline(src_stream)) != NULL) {
 		int (*decode_fn_ptr)(FILE * src, FILE * dst);
 		char *line_ptr;
 		FILE *dst_stream;

Modified: trunk/busybox/editors/patch.c
===================================================================
--- trunk/busybox/editors/patch.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/editors/patch.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -32,7 +32,7 @@
 
 	while (src_stream && (i < lines_count)) {
 		char *line;
-		line = bb_get_line_from_file(src_stream);
+		line = xmalloc_fgets(src_stream);
 		if (line == NULL) {
 			break;
 		}
@@ -96,7 +96,7 @@
 		ret = 0;
 	}
 
-	patch_line = bb_get_line_from_file(patch_file);
+	patch_line = xmalloc_fgets(patch_file);
 	while (patch_line) {
 		FILE *src_stream;
 		FILE *dst_stream;
@@ -115,14 +115,14 @@
 		 */
 		while (patch_line && strncmp(patch_line, "--- ", 4) != 0) {
 			free(patch_line);
-			patch_line = bb_get_line_from_file(patch_file);
+			patch_line = xmalloc_fgets(patch_file);
 		}
 
 		/* Extract the filename used before the patch was generated */
 		original_filename = extract_filename(patch_line, patch_level);
 		free(patch_line);
 
-		patch_line = bb_get_line_from_file(patch_file);
+		patch_line = xmalloc_fgets(patch_file);
 		if (strncmp(patch_line, "+++ ", 4) != 0) {
 			ret = 2;
 			bb_error_msg("Invalid patch");
@@ -166,7 +166,7 @@
 		printf("patching file %s\n", new_filename);
 
 		/* Handle each hunk */
-		patch_line = bb_get_line_from_file(patch_file);
+		patch_line = xmalloc_fgets(patch_file);
 		while (patch_line) {
 			unsigned int count;
 			unsigned int src_beg_line;
@@ -197,11 +197,11 @@
 			}
 			hunk_offset_start = src_cur_line;
 
-			while ((patch_line = bb_get_line_from_file(patch_file)) != NULL) {
+			while ((patch_line = xmalloc_fgets(patch_file)) != NULL) {
 				if ((*patch_line == '-') || (*patch_line == ' ')) {
 					char *src_line = NULL;
 					if (src_stream) {
-						src_line = bb_get_line_from_file(src_stream);
+						src_line = xmalloc_fgets(src_stream);
 						if (!src_line) {
 							hunk_error++;
 							break;

Modified: trunk/busybox/editors/sed.c
===================================================================
--- trunk/busybox/editors/sed.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/editors/sed.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -904,7 +904,7 @@
 						if (rfile) {
 							char *line;
 
-							while ((line = bb_get_chomped_line_from_file(rfile))
+							while ((line = xmalloc_getline(rfile))
 									!= NULL)
 								append(line);
 							xprint_and_close_file(rfile);
@@ -1099,7 +1099,7 @@
 	if (!opt_f) return;
 	add_files_link(opt_f->link);
 	cmdfile = xfopen(opt_f->data, "r");
-	while ((line = bb_get_chomped_line_from_file(cmdfile)) != NULL) {
+	while ((line = xmalloc_getline(cmdfile)) != NULL) {
 		add_cmd(line);
 		free(line);
 	}

Modified: trunk/busybox/findutils/grep.c
===================================================================
--- trunk/busybox/findutils/grep.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/findutils/grep.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -123,7 +123,7 @@
 	int idx = 0; /* used for iteration through the circular buffer */
 #endif /* ENABLE_FEATURE_GREP_CONTEXT */
 
-	while ((line = bb_get_chomped_line_from_file(file)) != NULL) {
+	while ((line = xmalloc_getline(file)) != NULL) {
 		llist_t *pattern_ptr = pattern_head;
 		grep_list_data_t * gl;
 
@@ -280,7 +280,7 @@
 		fopt = cur->link;
 		free(cur);
 		f = xfopen(ffile, "r");
-		while ((line = bb_get_chomped_line_from_file(f)) != NULL) {
+		while ((line = xmalloc_getline(f)) != NULL) {
 			llist_add_to(&pattern_head,
 				new_grep_list_data(line, PATTERN_MEM_A));
 		}

Modified: trunk/busybox/include/libbb.h
===================================================================
--- trunk/busybox/include/libbb.h	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/include/libbb.h	2006-10-12 22:43:20 UTC (rev 16375)
@@ -230,8 +230,9 @@
 extern long *find_pid_by_name( const char* pidName);
 extern long *pidlist_reverse(long *pidList);
 extern char *find_block_device(char *path);
-extern char *bb_get_line_from_file(FILE *file);
-extern char *bb_get_chomped_line_from_file(FILE *file);
+extern char *xmalloc_fgets(FILE *file);
+/* Chops off '\n' from the end, unlike fgets: */
+extern char *xmalloc_getline(FILE *file);
 extern char *bb_get_chunk_from_file(FILE *file, int *end);
 extern off_t bb_copyfd_size(int fd1, int fd2, off_t size);
 extern off_t bb_copyfd_eof(int fd1, int fd2);

Modified: trunk/busybox/libbb/get_line_from_file.c
===================================================================
--- trunk/busybox/libbb/get_line_from_file.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/libbb/get_line_from_file.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -48,7 +48,7 @@
 }
 
 /* Get line, including trailing \n if any */
-char *bb_get_line_from_file(FILE * file)
+char *xmalloc_fgets(FILE * file)
 {
 	int i;
 
@@ -56,7 +56,7 @@
 }
 
 /* Get line.  Remove trailing \n */
-char *bb_get_chomped_line_from_file(FILE * file)
+char *xmalloc_getline(FILE * file)
 {
 	int i;
 	char *c = bb_get_chunk_from_file(file, &i);

Modified: trunk/busybox/miscutils/dc.c
===================================================================
--- trunk/busybox/miscutils/dc.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/miscutils/dc.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -208,7 +208,7 @@
 		char *line   = NULL;
 		char *cursor = NULL;
 		char *token  = NULL;
-		while ((line = bb_get_chomped_line_from_file(stdin))) {
+		while ((line = xmalloc_getline(stdin))) {
 			cursor = line;
 			len = number_of_tokens(line);
 			for (i = 0; i < len; i++) {

Modified: trunk/busybox/miscutils/makedevs.c
===================================================================
--- trunk/busybox/miscutils/makedevs.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/miscutils/makedevs.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -95,7 +95,7 @@
 		printf("table=<stdin>\n");
 	}
 
-	while ((line = bb_get_chomped_line_from_file(table))) {
+	while ((line = xmalloc_getline(table))) {
 		char type;
 		unsigned int mode = 0755;
 		unsigned int major = 0;

Modified: trunk/busybox/networking/dnsd.c
===================================================================
--- trunk/busybox/networking/dnsd.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/networking/dnsd.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -138,7 +138,7 @@
 	char *r, *name;
 
  restart:
-	r = bb_get_line_from_file(fp);
+	r = xmalloc_fgets(fp);
 	if (!r)
 		return -1;
 	while (*r == ' ' || *r == '\t') {

Modified: trunk/busybox/networking/ifupdown.c
===================================================================
--- trunk/busybox/networking/ifupdown.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/networking/ifupdown.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -649,7 +649,7 @@
 
 	f = xfopen(filename, "r");
 
-	while ((buf = bb_get_chomped_line_from_file(f)) != NULL) {
+	while ((buf = xmalloc_getline(f)) != NULL) {
 		char *buf_ptr = buf;
 
 		firstword = next_word(&buf_ptr);

Modified: trunk/busybox/networking/nameif.c
===================================================================
--- trunk/busybox/networking/nameif.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/networking/nameif.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -91,7 +91,7 @@
 	} else {
 		ifh = xfopen(fname, "r");
 
-		while ((line = bb_get_line_from_file(ifh)) != NULL) {
+		while ((line = xmalloc_fgets(ifh)) != NULL) {
 			char *line_ptr;
 			size_t name_length;
 

Modified: trunk/busybox/shell/cmdedit.c
===================================================================
--- trunk/busybox/shell/cmdedit.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/shell/cmdedit.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -1205,7 +1205,7 @@
 	if (( fp = fopen ( fromfile, "r" ))) {
 
 		for ( hi = 0; hi < MAX_HISTORY; ) {
-			char * hl = bb_get_chomped_line_from_file(fp);
+			char * hl = xmalloc_getline(fp);
 			int l;
 
 			if(!hl)

Modified: trunk/busybox/util-linux/hexdump.c
===================================================================
--- trunk/busybox/util-linux/hexdump.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/util-linux/hexdump.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -21,7 +21,7 @@
 
 	fp = xfopen(name, "r");
 
-	while ((buf = bb_get_chomped_line_from_file(fp)) != NULL) {
+	while ((buf = xmalloc_getline(fp)) != NULL) {
 		p = skip_whitespace(buf);
 
 		if (*p && (*p != '#')) {

Modified: trunk/busybox/util-linux/mount.c
===================================================================
--- trunk/busybox/util-linux/mount.c	2006-10-12 22:42:33 UTC (rev 16374)
+++ trunk/busybox/util-linux/mount.c	2006-10-12 22:43:20 UTC (rev 16375)
@@ -186,7 +186,7 @@
 		f = fopen(filesystems[i], "r");
 		if (!f) continue;
 
-		while ((buf = bb_get_chomped_line_from_file(f)) != 0) {
+		while ((buf = xmalloc_getline(f)) != 0) {
 			if (!strncmp(buf, "nodev", 5) && isspace(buf[5]))
 				continue;
 			fs = buf;




More information about the busybox-cvs mailing list