This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
[PATCH] gdb.threads/linux-dp.exp: adjust to new gcc warning
- From: Michael Elizabeth Chastain <mec at shout dot net>
- To: gdb-patches at sources dot redhat dot com
- Date: Mon, 18 Feb 2002 15:44:13 -0600
- Subject: [PATCH] gdb.threads/linux-dp.exp: adjust to new gcc warning
gcc HEAD has a new warning:
warning: no return statement in function returning non-void
Unfortunately this breaks code in gdb.threads/linux-dp.c:
void *
philosopher (void *data)
{
...
for (;;)
{
...
}
}
I consider this to be a bug in gcc HEAD, and I've filed PR gcc/5725.
Meanwhile, I would like to change linux-dp.c so that it compiles.
This way we are testing linux thread support, which is more useful
than functioning as a test case for PR gcc/5725.
I tested this patch on target=native host=i686-pc-linux-gnulibc2.2:rh-7.2
with gcc 2.95.3, gcc 3.0.3, gcc 3.0.4-20020215, gcc gcc-3_0-branch, and
gcc HEAD, with both -gdwarf-2 and -gstabs+.
References:
Jason's message about the new warning:
http://gcc.gnu.org/ml/gcc-patches/2002-02/msg00034.html
My reply in the thread:
http://gcc.gnu.org/ml/gcc-patches/2002-02/msg00704.html
My gcc PR:
http://gcc.gnu.org/cgi-bin/gnatsweb.pl?database=gcc&cmd=view&pr=5725
Okay to apply?
Michael C
===
2002-02-18 Michael Chastain <mec@shout.net>
* gdb.threads/linux-dp.c (philosopher): Add a return statement
to placate gcc.
Index: gdb/testsuite/gdb.threads/linux-dp.c
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.threads/linux-dp.c,v
retrieving revision 1.1.1.1
diff -c -3 -p -r1.1.1.1 linux-dp.c
*** linux-dp.c 1999/10/19 02:46:48 1.1.1.1
--- linux-dp.c 2002/02/18 21:37:54
*************** philosopher (void *data)
*** 153,158 ****
--- 153,160 ----
pthread_mutex_unlock (&fork_mutex[(n + 1) % num_philosophers]);
random_delay ();
}
+
+ return (void *) 0;
}
int