This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch] -Wall patches
- From: Hui Zhu <teawater at gmail dot com>
- To: Aleksandar Ristovski <aristovski at qnx dot com>
- Cc: gdb-patches at sources dot redhat dot com
- Date: Sat, 8 Aug 2009 09:55:16 +0800
- Subject: Re: [patch] -Wall patches
- References: <h5fdom$ibb$1@ger.gmane.org>
2009/8/7 Aleksandar Ristovski <aristovski@qnx.com>:
> Hello,
>
>
> I have internally made changes necessary to get clean compile with -Wall
> compiler option. I tried to break down the patch to related files; I am
> intentionally not writing a changelog since most of the changes are
> mechanical and consist of removing unused variables, several unused
> functions, gdb_byte/char casting.
>
> I would not mind if respective maintainers looked at the patches for their
> files and committed the changes (otherwise it will be quite cumbersome and
> somewhat overwhelming for me to keep the patch applicable; not to mention
> change log and all).
>
>
> Thanks,
>
> --
> Aleksandar Ristovski
> QNX Software Systems
>
Thanks Aleksandar.
I will check in record.c patch under the obvious rule.
Hui
2009-08-08 Aleksandar Ristovski <aristovski@qnx.com>
Hui Zhu <teawater@gmail.com>
* record.c (record_store_registers): Remove unused variable.
Index: record.c
===================================================================
RCS file: /cvs/src/src/gdb/record.c,v
retrieving revision 1.10
diff -p -u -r1.10 record.c
--- record.c 4 Aug 2009 18:19:52 -0000 1.10
+++ record.c 8 Aug 2009 01:49:41 -0000
@@ -959,7 +959,6 @@ record_store_registers (struct target_op
if (RECORD_IS_REPLAY)
{
int n;
- struct cleanup *old_cleanups;
/* Let user choose if he wants to write register or not. */
if (regno < 0)