This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[commit] Fix -Wunused warning in dec-thread.c.
- From: Joel Brobecker <brobecker at adacore dot com>
- To: gdb-patches at sourceware dot org
- Date: Tue, 12 Jan 2010 11:56:11 +0400
- Subject: [commit] Fix -Wunused warning in dec-thread.c.
From: brobecke <brobecke@f8352e7e-cb20-0410-8ce7-b5d9e71c585c>
Hello,
The compiler complains that the result of a couple of *PTR++ is never
used. This is because the compiler is tricked by the way the pointer
was defined. For instance:
| dec_thread_count_gdb_threads (struct thread_info *ignored, void *context)
| {
| int *count = (int *) context;
The compiler complains that the result is never used for the following
line:
| *count++;
The result is stored for the caller to use, so it is in fact potentially
used, but elsewhere...
gdb/ChangeLog:
* dec-thread.c (dec_thread_count_gdb_threads)
(dec_thread_add_gdb_thread): Prevent -Wunused warning.
Checked in.
---
gdb/dec-thread.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/gdb/dec-thread.c b/gdb/dec-thread.c
index 5083221..be6db4c 100644
--- a/gdb/dec-thread.c
+++ b/gdb/dec-thread.c
@@ -352,7 +352,7 @@ dec_thread_count_gdb_threads (struct thread_info *ignored, void *context)
{
int *count = (int *) context;
- *count++;
+ (void) *count++; /* The cast to void is to prevent a -Wunused warning. */
return 0;
}
@@ -366,7 +366,7 @@ dec_thread_add_gdb_thread (struct thread_info *info, void *context)
struct thread_info ***listp = (struct thread_info ***) context;
**listp = info;
- *listp++;
+ (void) *listp++; /* The cast to void is to prevent a -Wunused warning. */
return 0;
}
--
1.6.3.3