# HG changeset patch
# User reimar
# Date 1267361652 0
# Node ID 7c412873705e8901c5f1453283a79c6a87bec206
# Parent  abec4bb780e53807ab4ff67f9b9a6d13d5eb298d
Simplify handling of 0-termination in stream_read_line

diff -r abec4bb780e5 -r 7c412873705e stream/stream.h
--- a/stream/stream.h	Sun Feb 28 11:30:35 2010 +0000
+++ b/stream/stream.h	Sun Feb 28 12:54:12 2010 +0000
@@ -268,6 +268,8 @@
 inline static unsigned char* stream_read_line(stream_t *s,unsigned char* mem, int max) {
   int len;
   unsigned char* end,*ptr = mem;
+  if (max < 1) return NULL;
+  max--; // reserve one for 0-termination
   do {
     len = s->buf_len-s->buf_pos;
     // try to fill the buffer
@@ -276,8 +278,8 @@
         (len = s->buf_len-s->buf_pos) <= 0)) break;
     end = (unsigned char*) memchr((void*)(s->buffer+s->buf_pos),'\n',len);
     if(end) len = end - (s->buffer+s->buf_pos) + 1;
-    if(len > 0 && max > 1) {
-      int l = len > max-1 ? max-1 : len;
+    if(len > 0 && max > 0) {
+      int l = len > max ? max : len;
       memcpy(ptr,s->buffer+s->buf_pos,l);
       max -= l;
       ptr += l;
@@ -285,7 +287,7 @@
     s->buf_pos += len;
   } while(!end);
   if(s->eof && ptr == mem) return NULL;
-  if(max > 0) ptr[0] = 0;
+  ptr[0] = 0;
   return mem;
 }