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

Vitaliy dfs at sdsd
Sun Jan 4 16:54:00 CST 2009


titon barua wrote:
> ---
>      /* we don't care about the success or failure of this call */
> -    write(*(This->fd), &event, sizeof(event));
> +    /* print errors if they occur */
> +    if (write(*(This->fd), &event, sizeof(event)) == -1) perror ( NULL );
What's the point of your patch? What part of the comment isn't clear to you?
Also don't use perror. Wine has ERR(), WARN(), TRACE() macros for that.

Vitaliy



More information about the wine-devel mailing list