aboutsummaryrefslogtreecommitdiff
path: root/toys
diff options
context:
space:
mode:
authorElliott Hughes <enh@google.com>2019-08-10 09:44:58 -0700
committerRob Landley <rob@landley.net>2019-08-12 04:44:00 -0500
commit30cb3aa0e33daa20d6adb0825979f79e615c9c47 (patch)
tree8134f1634c5640ffd196895fa4e4f11e8af10364 /toys
parent9dc24d994efc3837e248ef8db9b3976e8775235e (diff)
downloadtoybox-30cb3aa0e33daa20d6adb0825979f79e615c9c47.tar.gz
sort: move off get_rawline.
This was the last user of get_rawline, which lets us remove it.
Diffstat (limited to 'toys')
-rw-r--r--toys/posix/sort.c47
1 files changed, 27 insertions, 20 deletions
diff --git a/toys/posix/sort.c b/toys/posix/sort.c
index 9d2f2276..4b3fe24d 100644
--- a/toys/posix/sort.c
+++ b/toys/posix/sort.c
@@ -63,6 +63,7 @@ GLOBALS(
void *key_list;
int linecount;
char **lines;
+ char *name;
)
// The sort types are n, g, and M.
@@ -275,31 +276,37 @@ static int compare_keys(const void *xarg, const void *yarg)
return retval * ((flags&FLAG_r) ? -1 : 1);
}
-// Callback from loopfiles to handle input files.
-static void sort_read(int fd, char *name)
+// Read each line from file, appending to a big array.
+static void sort_lines(char **pline, long len)
{
- // Read each line from file, appending to a big array.
-
- for (;;) {
- char * line = FLAG(z) ? get_rawline(fd, NULL, 0) : get_line(fd);
+ char * line;
- if (!line) break;
+ if (!pline) return;
+ line = *pline;
+ if (!FLAG(z) && len && line[len-1]=='\n') line[--len] = 0;
+ *pline = NULL;
- // handle -c here so we don't allocate more memory than necessary.
- if (FLAG(c)) {
- int j = FLAG(u) ? -1 : 0;
+ // handle -c here so we don't allocate more memory than necessary.
+ if (FLAG(c)) {
+ int j = FLAG(u) ? -1 : 0;
- if (TT.lines && compare_keys((void *)&TT.lines, &line)>j)
- error_exit("%s: Check line %d\n", name, TT.linecount);
- free(TT.lines);
- TT.lines = (char **)line;
- } else {
- if (!(TT.linecount&63))
- TT.lines = xrealloc(TT.lines, sizeof(char *)*(TT.linecount+64));
- TT.lines[TT.linecount] = line;
- }
- TT.linecount++;
+ if (TT.lines && compare_keys((void *)&TT.lines, &line)>j)
+ error_exit("%s: Check line %d\n", TT.name, TT.linecount);
+ free(TT.lines);
+ TT.lines = (char **)line;
+ } else {
+ if (!(TT.linecount&63))
+ TT.lines = xrealloc(TT.lines, sizeof(char *)*(TT.linecount+64));
+ TT.lines[TT.linecount] = line;
}
+ TT.linecount++;
+}
+
+// Callback from loopfiles to handle input files.
+static void sort_read(int fd, char *name)
+{
+ TT.name = name;
+ do_lines(fd, FLAG(z) ? '\0' : '\n', sort_lines);
}
void sort_main(void)