Re: CVS commit: src/usr.bin/audiocfg

[christos%astron.com@localhost (Christos Zoulas)]

In article <20100903192037.C6AA5175D0%cvs.netbsd.org@localhost>,
Jared D. McNeill <source-changes-d%NetBSD.org@localhost> 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