In article <20100903192037.c6aa517...@cvs.netbsd.org>, Jared D. McNeill <source-changes-d@NetBSD.org> wrote:
[...]
> if (wlen > buflen)
> wlen = buflen;
>- write(adev->fd, (char *)buf + off, wlen);
>+ wlen = write(adev->fd, (char *)buf + off, wlen);
>+ if (wlen == -1) {
>+ perror("write");
>+ rv = -1;
>+ goto done;
>+ }
Should we make perror() produce a warning so people will stop using it?
Please use warn{,x}() instead, so we get the program name printed!
christos
