vcs-svn: allow input from file descriptor
authorJonathan Nieder <jrnieder@gmail.com>
Mon, 3 Jan 2011 03:09:38 +0000 (21:09 -0600)
committerJonathan Nieder <jrnieder@gmail.com>
Sat, 26 Feb 2011 10:59:37 +0000 (04:59 -0600)
Based-on-patch-by: David Barr <david.barr@cordelta.com>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
t/t0081-line-buffer.sh
test-line-buffer.c
vcs-svn/line_buffer.c
vcs-svn/line_buffer.h
vcs-svn/line_buffer.txt

index a8eeb206457be723ac35c35e557f45967650dc89..550fad0823a8d27cf1d49b1c55154ea7ce69de02 100755 (executable)
@@ -131,6 +131,15 @@ test_expect_success PIPE,EXPENSIVE 'longer read (around 65536 bytes)' '
        long_read_test 65536
 '
 
+test_expect_success 'read from file descriptor' '
+       rm -f input &&
+       echo hello >expect &&
+       echo hello >input &&
+       echo copy 6 |
+       test-line-buffer "&4" 4<input >actual &&
+       test_cmp expect actual
+'
+
 test_expect_success 'buffer_read_string copes with null byte' '
        >expect &&
        q_to_nul <<-\EOF | test-line-buffer >actual &&
index 19bf2d4403d03498b5c246367e59265535d2f90c..25b20b93fd4a1113629c99bd5a53dfee965119c1 100644 (file)
@@ -69,13 +69,18 @@ int main(int argc, char *argv[])
        else if (argc == 2)
                filename = argv[1];
        else
-               usage("test-line-buffer [file] < script");
+               usage("test-line-buffer [file | &fd] < script");
 
        if (buffer_init(&stdin_buf, NULL))
                die_errno("open error");
        if (filename) {
-               if (buffer_init(&file_buf, filename))
-                       die_errno("error opening %s", filename);
+               if (*filename == '&') {
+                       if (buffer_fdinit(&file_buf, strtouint32(filename + 1)))
+                               die_errno("error opening fd %s", filename + 1);
+               } else {
+                       if (buffer_init(&file_buf, filename))
+                               die_errno("error opening %s", filename);
+               }
                input = &file_buf;
        }
 
index 37ec56e5be46e88bcd9a2f41d70170a642297b9f..e29a81a536de5a6e68a5b434a330f23a96f304bb 100644 (file)
@@ -17,6 +17,14 @@ int buffer_init(struct line_buffer *buf, const char *filename)
        return 0;
 }
 
+int buffer_fdinit(struct line_buffer *buf, int fd)
+{
+       buf->infile = fdopen(fd, "r");
+       if (!buf->infile)
+               return -1;
+       return 0;
+}
+
 int buffer_deinit(struct line_buffer *buf)
 {
        int err;
index 0a59c73e8b858a44e22620c8b1cb5c4c02c5343e..630d83c31a1e9b35a20452f0b687f7b364ee81a1 100644 (file)
@@ -13,6 +13,7 @@ struct line_buffer {
 #define LINE_BUFFER_INIT {"", STRBUF_INIT, NULL}
 
 int buffer_init(struct line_buffer *buf, const char *filename);
+int buffer_fdinit(struct line_buffer *buf, int fd);
 int buffer_deinit(struct line_buffer *buf);
 char *buffer_read_line(struct line_buffer *buf);
 char *buffer_read_string(struct line_buffer *buf, uint32_t len);
index f8eaa4dd8c2508c6e0c5b9b46649dc7232e50116..4e8fb719c1745de96a64e7fd4cb49fd2538811aa 100644 (file)
@@ -27,10 +27,11 @@ resources.
 Functions
 ---------
 
-`buffer_init`::
-       Open the named file for input.  If filename is NULL,
-       start reading from stdin.  On failure, returns -1 (with
-       errno indicating the nature of the failure).
+`buffer_init`, `buffer_fdinit`::
+       Open the named file or file descriptor for input.
+       buffer_init(buf, NULL) prepares to read from stdin.
+       On failure, returns -1 (with errno indicating the nature
+       of the failure).
 
 `buffer_deinit`::
        Stop reading from the current file (closing it unless