Commit: c49e7f76afc4507257d66f1c72fabd83c2d6ef18 - binutils_os216 (git)

GNU Binutils with patches for OS216

Commit MetaInfo

• Revisión: c49e7f76afc4507257d66f1c72fabd83c2d6ef18 (tree)
• Tiempo: 2013-04-20 02:48:10
• Autor: Pedro Alves <palves@redh...>
• Commiter: Pedro Alves

Log Message

Make GDB -Wpointer-sign clean on MinGW too.

This is the remaining issue impeding GDB to build with "-Wpointer-sign
-Werror" with Fedora 17's i686-w64-mingw32 cross toolchain.

../../src/gdb/ser-tcp.c: In function 'net_read_prim':
../../src/gdb/ser-tcp.c:341:3: error: pointer targets in passing argument 2 of 'recv' differ in signedness [-Werror=pointer-sign]
In file included from ../../src/gdb/serial.h:23:0,

from ../../src/gdb/ser-tcp.c:21:

/usr/i686-w64-mingw32/sys-root/mingw/include/winsock2.h:983:34: note: expected 'char *' but argument is of type 'unsigned char *'

gdb/
2013-04-19 Pedro Alves <[e-mail]>

* ser-tcp.c (net_read_prim): Cast second argument to recv to
'void *'.

Cambiar Resumen

• modified: gdb/ChangeLog
• modified: gdb/ser-tcp.c

Diferencia incremental

--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,10 @@
 2013-04-19  Pedro Alves  <[e-mail]>
 
+	* ser-tcp.c (net_read_prim): Cast second argument to recv to
+	'void *'.
+
[phone]  Pedro Alves  <[e-mail]>
+
 	* monitor.c (monitor_write_memory, monitor_write_memory_bytes):
 	Change type of 'myaddr' parameter to gdb_byte pointer.
 	(monitor_write_memory_longlongs): Likewise.  Cast 'myaddr' pointer

--- a/gdb/ser-tcp.c
+++ b/gdb/ser-tcp.c
@@ -338,7 +338,10 @@ net_close (struct serial *scb)
 int
 net_read_prim (struct serial *scb, size_t count)
 {
-  return recv (scb->fd, scb->buf, count, 0);
+  /* Need to cast to silence -Wpointer-sign on MinGW, as Winsock's
+     'recv' takes 'char *' as second argument, while 'scb->buf' is
+     'unsigned char *'.  */
+  return recv (scb->fd, (void *) scb->buf, count, 0);
 }
 
 int