On Fri, 2006-06-16 at 11:18 -0700, Paul Eggert wrote:
> Simon Josefsson <[EMAIL PROTECTED]> writes:
>
> > + if (buf)
> > + buf[size] = '\0';
>
> The file reading code returns NULL if the file was empty? But the
> comments makes it sound like it will return xstrdup (""), and that
> would be more consistent. You might also add a test case, to read
> from /dev/null.
If I were using a function like this, I'd expect NULL on error (and
errno set according to fopen), and an empty string if the file is empty.
Aaron
