[pushed] value.c: Remove unused variables

Message ID 1526565190-7873-1-git-send-email-simon.marchi@ericsson.com
State New, archived
Headers

Commit Message

Simon Marchi May 17, 2018, 1:53 p.m. UTC
  Obvious patch to remove unused local variables (found by adding
-Wunused).  I didn't touch this one in value_fetch_lazy, because
check_typedef could have a desired side-effect.

  3743  struct type *type = check_typedef (value_type (val));

gdb/ChangeLog:

	* value.c (release_value): Remove unused variable.
	(record_latest_value): Likewise.
	(access_value_history): Likewise.
	(preserve_values): Likewise.
---
 gdb/ChangeLog | 7 +++++++
 gdb/value.c   | 6 ------
 2 files changed, 7 insertions(+), 6 deletions(-)
  

Patch

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 4ae46c1..94c351b 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,10 @@ 
+2018-05-17  Simon Marchi  <simon.marchi@ericsson.com>
+
+	* value.c (release_value): Remove unused variable.
+	(record_latest_value): Likewise.
+	(access_value_history): Likewise.
+	(preserve_values): Likewise.
+
 2018-05-17  Tom Tromey  <tom@tromey.com>
 
 	* extension.h (struct ext_lang_type_printers) <py_type_printers>:
diff --git a/gdb/value.c b/gdb/value.c
index eb28121..aa6ca43 100644
--- a/gdb/value.c
+++ b/gdb/value.c
@@ -1614,8 +1614,6 @@  value_free_to_mark (const struct value *mark)
 value_ref_ptr
 release_value (struct value *val)
 {
-  struct value *v;
-
   if (val == nullptr)
     return value_ref_ptr ();
 
@@ -1789,8 +1787,6 @@  set_value_component_location (struct value *component,
 int
 record_latest_value (struct value *val)
 {
-  int i;
-
   /* We don't want this value to have anything to do with the inferior anymore.
      In particular, "set $1 = 50" should not affect the variable from which
      the value was taken, and fast watchpoints should be able to assume that
@@ -1812,7 +1808,6 @@  record_latest_value (struct value *val)
 struct value *
 access_value_history (int num)
 {
-  int i;
   int absnum = num;
 
   if (absnum <= 0)
@@ -2507,7 +2502,6 @@  preserve_values (struct objfile *objfile)
 {
   htab_t copied_types;
   struct internalvar *var;
-  int i;
 
   /* Create the hash table.  We allocate on the objfile's obstack, since
      it is soon to be deleted.  */