diff options
| author |  Ramsay Jones <ramsay@ramsay1.demon.co.uk> | 2013-04-27 20:17:14 +0100 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2013-04-28 12:27:04 -0700 |
| commit | 1c31596a4b029e51cef1c520f32a3ecfbdea3c0a (patch) | |
| tree | b812c9e1a9002fede759d71fa496cfedfd7659d8 | |
| parent | compat/win32/pthread.c: Fix a sparse warning (diff) | |
| download | tgif-1c31596a4b029e51cef1c520f32a3ecfbdea3c0a.tar.xz | |
compat/poll/poll.c: Fix a sparse warning
Sparse issues an 'Using plain integer as NULL pointer' warning when
passing the constant '0' as the second parameter in the call to the
WSAEventSelect() function. The function parameter has a pointer type
(WSAEVENT, aka HANDLE, aka void *) so that, in order to suppress the
warning, we simply pass NULL for that parameter in the function call
expression.
Signed-off-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | compat/poll/poll.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/compat/poll/poll.c b/compat/poll/poll.c index 7d226ecb29..44103103a4 100644 --- a/compat/poll/poll.c +++ b/compat/poll/poll.c @@ -576,7 +576,7 @@ restart: { /* It's a socket. */ WSAEnumNetworkEvents ((SOCKET) h, NULL, &ev); - WSAEventSelect ((SOCKET) h, 0, 0); + WSAEventSelect ((SOCKET) h, NULL, 0); /* If we're lucky, WSAEnumNetworkEvents already provided a way to distinguish FD_READ and FD_ACCEPT; this saves a recv later. */ |