dlls/dinput/effect_linuxinput.c: Check return value of 'write'. (bug # 16413)

Dan Kegel dank at kegel.com
Mon Jan 5 08:42:21 CST 2009


Damjan wrote:
>ssize_t ignored;
>...
>ignored = write(*(This->fd), &event, sizeof(event));
>
>You could make the ignored variable static, maybe even global.

Please don't.  We don't want to ignore the errors,
we want to handle them.  The changes you're
proposing are harmful to readability.  Better
to ignore the compiler warnings than to
paper them over like that.



More information about the wine-devel mailing list