summaryrefslogtreecommitdiff
path: root/coreutils
diff options
context:
space:
mode:
authorJohn Beppu2000-04-17 04:22:09 +0000
committerJohn Beppu2000-04-17 04:22:09 +0000
commit5a728cfdfeaa0c6db8485bec230e24b3ca03806b (patch)
treedb9e84821fb8742d8c10636388db640f1f763c73 /coreutils
parent3becdfc31635bec63b6cbefde148d9bd3f3678a1 (diff)
downloadbusybox-5a728cfdfeaa0c6db8485bec230e24b3ca03806b.zip
busybox-5a728cfdfeaa0c6db8485bec230e24b3ca03806b.tar.gz
+ in the interest of robustness, I added
utility.c :: cstring_alloc() utility.c :: cstring_lineFromFile() /* they're at the bottom */ so that I could read in lines of arbitrary length from FILE*s (instead of using fgets(huge_ass_buffer,...)). + I tested it out on sort, and it seems to be fine.
Diffstat (limited to 'coreutils')
-rw-r--r--coreutils/sort.c32
1 files changed, 6 insertions, 26 deletions
diff --git a/coreutils/sort.c b/coreutils/sort.c
index 4301f43..49eb4fd 100644
--- a/coreutils/sort.c
+++ b/coreutils/sort.c
@@ -54,7 +54,6 @@ typedef struct Line {
typedef struct {
int len; /* number of Lines */
Line **sorted; /* array fed to qsort */
-
Line *head; /* head of List */
Line *current; /* current Line */
} List;
@@ -71,36 +70,23 @@ static const int max = 1024;
static Line *line_alloc()
{
Line *self;
-
self = malloc(1 * sizeof(Line));
return self;
}
-/* Initialize Line with string */
-static Line *line_init(Line * self, const char *string)
-{
- self->data = malloc((strlen(string) + 1) * sizeof(char));
-
- if (self->data == NULL) {
- return NULL;
- }
- strcpy(self->data, string);
- self->next = NULL;
- return self;
-}
-
/* Construct Line from FILE* */
static Line *line_newFromFile(FILE * src)
{
- char buffer[max];
Line *self;
+ char *cstring = NULL;
- if (fgets(buffer, max, src)) {
+ if ((cstring = cstring_lineFromFile(src))) {
self = line_alloc();
if (self == NULL) {
return NULL;
}
- line_init(self, buffer);
+ self->data = cstring;
+ self->next = NULL;
return self;
}
return NULL;
@@ -177,7 +163,7 @@ static List *list_insert(List * self, Line * line)
self->head = line;
self->current = line;
- /* all subsequent insertions */
+ /* all subsequent insertions */
} else {
self->current->next = line;
self->current = line;
@@ -241,12 +227,6 @@ static void list_release(List * self)
}
-/*
- * I need a list
- * to insert lines into
- * then I need to sort this list
- * and finally print it
- */
int sort_main(int argc, char **argv)
{
@@ -320,4 +300,4 @@ int sort_main(int argc, char **argv)
exit(0);
}
-/* $Id: sort.c,v 1.14 2000/04/15 16:34:54 erik Exp $ */
+/* $Id: sort.c,v 1.15 2000/04/17 04:22:09 beppu Exp $ */