Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [patch] cleanup: Wunused target.c
@ 2013-01-31  4:25 Aleksandar Ristovski
  2013-01-31 16:15 ` Tom Tromey
  0 siblings, 1 reply; 3+ messages in thread
From: Aleksandar Ristovski @ 2013-01-31  4:25 UTC (permalink / raw)
  To: gdb-patches

[-- Attachment #1: Type: text/plain, Size: 255 bytes --]

Hello,

This is removing unused variables and allows compiling with -Wunused.

Thanks,

Aleksandar Ristovski
QNX Software Systems


ChangeLog:

  Aleksandar Ristovski  <aristovski@qnx.com>

         * target.c (target_read_string): Remove unused ORIGLEN.

[-- Attachment #2: Wunused-target-201301302122.patch --]
[-- Type: text/x-patch, Size: 751 bytes --]

Index: gdb/target.c
===================================================================
RCS file: /cvs/src/src/gdb/target.c,v
retrieving revision 1.318
diff -u -p -r1.318 target.c
--- gdb/target.c	1 Jan 2013 06:32:52 -0000	1.318
+++ gdb/target.c	30 Jan 2013 22:25:16 -0000
@@ -1220,7 +1220,7 @@ target_translate_tls_address (struct obj
 int
 target_read_string (CORE_ADDR memaddr, char **string, int len, int *errnop)
 {
-  int tlen, origlen, offset, i;
+  int tlen, offset, i;
   gdb_byte buf[4];
   int errcode = 0;
   char *buffer;
@@ -1235,8 +1235,6 @@ target_read_string (CORE_ADDR memaddr, c
   buffer = xmalloc (buffer_allocated);
   bufptr = buffer;
 
-  origlen = len;
-
   while (len > 0)
     {
       tlen = MIN (len, 4 - (memaddr & 3));


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [patch] cleanup: Wunused target.c
  2013-01-31  4:25 [patch] cleanup: Wunused target.c Aleksandar Ristovski
@ 2013-01-31 16:15 ` Tom Tromey
  2013-01-31 16:33   ` Aleksandar Ristovski
  0 siblings, 1 reply; 3+ messages in thread
From: Tom Tromey @ 2013-01-31 16:15 UTC (permalink / raw)
  To: Aleksandar Ristovski; +Cc: gdb-patches

>>>>> "Aleksandar" == Aleksandar Ristovski <aristovski@qnx.com> writes:

Aleksandar>         * target.c (target_read_string): Remove unused ORIGLEN.

Ok.

Tom


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [patch] cleanup: Wunused target.c
  2013-01-31 16:15 ` Tom Tromey
@ 2013-01-31 16:33   ` Aleksandar Ristovski
  0 siblings, 0 replies; 3+ messages in thread
From: Aleksandar Ristovski @ 2013-01-31 16:33 UTC (permalink / raw)
  To: Tom Tromey; +Cc: gdb-patches

On 13-01-31 11:14 AM, Tom Tromey wrote:
>>>>>> "Aleksandar" == Aleksandar Ristovski <aristovski@qnx.com> writes:
>
> Aleksandar>         * target.c (target_read_string): Remove unused ORIGLEN.
>
> Ok.
>
> Tom
>

Committed: http://sourceware.org/ml/gdb-cvs/2013-01/msg00200.html


Thank you,

Aleksandar Ristovski
QNX Software Systems


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-01-31 16:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-01-31  4:25 [patch] cleanup: Wunused target.c Aleksandar Ristovski
2013-01-31 16:15 ` Tom Tromey
2013-01-31 16:33   ` Aleksandar Ristovski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox