From 58c1b36c673c88f13fe0cf3844fef643c35e8318 Mon Sep 17 00:00:00 2001 From: Pedro Alves Date: Fri, 10 Jul 2015 19:47:17 +0100 Subject: gdbserver/event-loop.c: type mismatch MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Building with C++ catches a buglet here: ../../../src/gdb/gdbserver/event-loop.c:205:19: warning: invalid conversion from ‘gdb_client_data {aka void*}’ to ‘void**’ [-fpermissive] event_ptr->data = data; ^ This works in practice because gdb_client_data is a pointer already (hence in C we get an implicit conversion), and nothing deferences the pointer. It's passed from client at event registration/creation time, only to pass straight back to client callback. Well, that and nothing in gdbserver uses the event data anyway. gdb/gdbserver/ChangeLog: 2015-07-10 Pedro Alves * event-loop.c (struct callback_event) : Change type to gdb_client_data instance instead of gdb_client_data pointer. (append_callback_event): Adjust. --- gdb/gdbserver/event-loop.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'gdb/gdbserver/event-loop.c') diff --git a/gdb/gdbserver/event-loop.c b/gdb/gdbserver/event-loop.c index 08a503f..d1825ae 100644 --- a/gdb/gdbserver/event-loop.c +++ b/gdb/gdbserver/event-loop.c @@ -130,7 +130,7 @@ struct callback_event { int id; callback_handler_func *proc; - gdb_client_data *data; + gdb_client_data data; struct callback_event *next; }; @@ -249,7 +249,7 @@ process_callback (void) if (event_ptr != NULL) { callback_handler_func *proc = event_ptr->proc; - gdb_client_data *data = event_ptr->data; + gdb_client_data data = event_ptr->data; /* Remove the event before calling PROC, more events may get added by PROC. */ -- cgit v1.1