# HG changeset patch
# User Eric Blake <eblake@redhat.com>
# Date 1304025886 21600
# Node ID 2482350603ac726e544880efd7ea5831376ee7d3
# Parent  5c448b1cba392ad1b97b2e54778cc7282a6d6243
getsockopt: avoid compiler warning

getsockopt.c: In function 'rpl_getsockopt':
getsockopt.c:51:7: warning: passing argument 4 of 'getsockopt' from
incompatible pointer type
c:\dev\msys_setup\mingw\bin\../lib/gcc/mingw32/4.5.0/../../../../include/winsock2.h:543:32:
note: expected 'char *' but argument is of type 'int *'

A simple cast works around the mingw signature lameness.  If only
they had picked void* instead of char*, like POSIX says...

* lib/getsockopt.c (rpl_getsockopt): Add a cast for mingw.
Reported by Matthias Bolte.

Signed-off-by: Eric Blake <eblake@redhat.com>

diff --git a/ChangeLog b/ChangeLog
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
 2011-04-28  Eric Blake  <eblake@redhat.com>
 
+	getsockopt: avoid compiler warning
+	* lib/getsockopt.c (rpl_getsockopt): Add a cast for mingw.
+	Reported by Matthias Bolte.
+
 	tests: drop unused link dependency
 	* modules/areadlinkat-tests (Makefile.am): Drop stale LDADD.
 	* modules/dirent-safer-tests (Makefile.am): Likewise.
diff --git a/lib/getsockopt.c b/lib/getsockopt.c
--- a/lib/getsockopt.c
+++ b/lib/getsockopt.c
@@ -46,7 +46,8 @@
       int milliseconds_len = sizeof (int);
       struct timeval tv;
       size_t n;
-      r = getsockopt (sock, level, optname, &milliseconds, &milliseconds_len);
+      r = getsockopt (sock, level, optname, (char *) &milliseconds,
+                      &milliseconds_len);
       tv.tv_sec = milliseconds / 1000;
       tv.tv_usec = (milliseconds - 1000 * tv.tv_sec) * 1000;
       n = sizeof (struct timeval);