# HG changeset patch
# User Elie De Brauwer <eliedebrauwer@gmail.com>
# Date 1356173411 -3600
# Node ID 54d248b907edb759f266088a55360ba6ec159ceb
# Parent  4094f14c47a24967b38208a1ca71f42637b44d20
Fix tac to handle the "abc\ndef" case properly

diff -r 4094f14c47a2 -r 54d248b907ed scripts/test/tac.test
--- a/scripts/test/tac.test	Sun Dec 23 00:37:42 2012 -0600
+++ b/scripts/test/tac.test	Sat Dec 22 11:50:11 2012 +0100
@@ -17,8 +17,7 @@
         "tac file1 notfound file2 2>stderr && echo ok ; tac stderr; rm stderr" \
         "one-B\none-A\ntwo-B\ntwo-A\ntac: notfound: No such file or directory\n" "" ""
 
-# echo -ne "abc\ndef" | tac actually gives "defabc\n"
-testing "tac no trailing newline" "tac -" "def\nabc\n" "" "abc\ndef"
+testing "tac no trailing newline" "tac -" "defabc\n" "" "abc\ndef"
 
 # xputs used by tac does not propagate this error condition properly. 
 #testing "tac > /dev/full" \
diff -r 4094f14c47a2 -r 54d248b907ed toys/other/tac.c
--- a/toys/other/tac.c	Sun Dec 23 00:37:42 2012 -0600
+++ b/toys/other/tac.c	Sat Dec 22 11:50:11 2012 +0100
@@ -23,8 +23,9 @@
   // Read in lines
   for (;;) {
     struct arg_list *temp;
+    long len;
 
-    if (!(c = get_line(fd))) break;
+    if (!(c = get_rawline(fd, &len, '\n'))) break;
 
     temp = xmalloc(sizeof(struct arg_list));
     temp->next = list;
@@ -35,7 +36,7 @@
   // Play them back.
   while (list) {
     struct arg_list *temp = list->next;
-    xputs(list->arg);
+    xprintf("%s", list->arg);
     free(list->arg);
     free(list);
     list = temp;