diff options
author | Pedro Alves <palves@redhat.com> | 2015-11-17 15:17:45 +0000 |
---|---|---|
committer | Pedro Alves <palves@redhat.com> | 2015-11-17 15:20:03 +0000 |
commit | 69e976f8cc902adc5ee8aec24e4a208f4afd51b3 (patch) | |
tree | d3817f643cfdc21038d23bec2da966e066ad4238 /gdb/ser-tcp.c | |
parent | c3de4d92dfd0a1fa8d32a00bbb99de2a4ee0e140 (diff) | |
download | gdb-69e976f8cc902adc5ee8aec24e4a208f4afd51b3.zip gdb-69e976f8cc902adc5ee8aec24e4a208f4afd51b3.tar.gz gdb-69e976f8cc902adc5ee8aec24e4a208f4afd51b3.tar.bz2 |
[C++/mingw] ser-tcp.c casts
Fixes a few errors like these:
../../src/gdb/ser-tcp.c: In function 'int net_open(serial*, const char*)':
../../src/gdb/ser-tcp.c:286:73: error: invalid conversion from 'void*' to 'char*' [-fpermissive]
res = getsockopt (scb->fd, SOL_SOCKET, SO_ERROR, (void *) &err, &len);
^
gdb/ChangeLog:
2015-11-17 Pedro Alves <palves@redhat.com>
* ser-tcp.c (net_open) : Cast getsockopt argument to char *
instead of void *. Update comment.
(net_read_prim): Cast recv argument to char * instead of void *.
(net_write_prim): Cast send argument to char *. Add comment.
Diffstat (limited to 'gdb/ser-tcp.c')
-rw-r--r-- | gdb/ser-tcp.c | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/gdb/ser-tcp.c b/gdb/ser-tcp.c index ce40b61..36196f3 100644 --- a/gdb/ser-tcp.c +++ b/gdb/ser-tcp.c @@ -280,10 +280,10 @@ net_open (struct serial *scb, const char *name) len = sizeof (err); /* On Windows, the fourth parameter to getsockopt is a "char *"; - on UNIX systems it is generally "void *". The cast to "void *" - is OK everywhere, since in C "void *" can be implicitly - converted to any pointer type. */ - res = getsockopt (scb->fd, SOL_SOCKET, SO_ERROR, (void *) &err, &len); + on UNIX systems it is generally "void *". The cast to "char *" + is OK everywhere, since in C++ any data pointer type can be + implicitly converted to "void *". */ + res = getsockopt (scb->fd, SOL_SOCKET, SO_ERROR, (char *) &err, &len); if (res < 0 || err) { /* Maybe the target still isn't ready to accept the connection. */ @@ -342,13 +342,17 @@ net_read_prim (struct serial *scb, size_t count) /* 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); + return recv (scb->fd, (char *) scb->buf, count, 0); } int net_write_prim (struct serial *scb, const void *buf, size_t count) { - return send (scb->fd, buf, count, 0); + /* On Windows, the second parameter to send is a "const char *"; on + UNIX systems it is generally "const void *". The cast to "const + char *" is OK everywhere, since in C++ any data pointer type can + be implicitly converted to "const void *". */ + return send (scb->fd, (const char *) buf, count, 0); } int |