[binutils-gdb] Cleanup value_fetch_lazy's comment and return value

[Simon Marchi <simark at sourceware dot org>]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=a844296a982bdff7c14dab355de0fca75486add6

commit a844296a982bdff7c14dab355de0fca75486add6
Author: Simon Marchi <simon.marchi@ericsson.com>
Date:   Mon Jul 6 13:04:11 2015 -0400

    Cleanup value_fetch_lazy's comment and return value
    
    The comment for value_fetch_lazy seems outdated. It says that it's only
    called from the value_contents and value_contents_all (macros!), which
    is not true.  Also, the return value seems useless now, despite what the
    comment says.
    
    gdb/ChangeLog:
    
    	* value.c (value_fetch_lazy): Update comment, change return
    	value to void.
    	* value.h (value_fetch_lazy): Change return value to void.

Diff:
---
 gdb/ChangeLog |  6 ++++++
 gdb/value.c   | 17 +++++------------
 gdb/value.h   |  2 +-
 3 files changed, 12 insertions(+), 13 deletions(-)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 4636653..29b7da7 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2015-07-06  Simon Marchi  <simon.marchi@ericsson.com>
+
+	* value.c (value_fetch_lazy): Update comment, change return
+	value to void.
+	* value.h (value_fetch_lazy): Change return value to void.
+
 2015-07-06  Andrew Burgess  <andrew.burgess@embecosm.com>
 
 	* tui/tui-data.c (tui_partial_win_by_name): Window name is const.
diff --git a/gdb/value.c b/gdb/value.c
index 86d7d00..a01f390 100644
--- a/gdb/value.c
+++ b/gdb/value.c
@@ -3781,21 +3781,15 @@ value_initialized (struct value *val)
   return val->initialized;
 }
 
-/* Called only from the value_contents and value_contents_all()
-   macros, if the current data for a variable needs to be loaded into
-   value_contents(VAL).  Fetches the data from the user's process, and
-   clears the lazy flag to indicate that the data in the buffer is
-   valid.
+/* Load the actual content of a lazy value.  Fetch the data from the
+   user's process and clear the lazy flag to indicate that the data in
+   the buffer is valid.
 
    If the value is zero-length, we avoid calling read_memory, which
    would abort.  We mark the value as fetched anyway -- all 0 bytes of
-   it.
+   it.  */
 
-   This function returns a value because it is used in the
-   value_contents macro as part of an expression, where a void would
-   not work.  The value is ignored.  */
-
-int
+void
 value_fetch_lazy (struct value *val)
 {
   gdb_assert (value_lazy (val));
@@ -3947,7 +3941,6 @@ value_fetch_lazy (struct value *val)
     internal_error (__FILE__, __LINE__, _("Unexpected lazy value type."));
 
   set_value_lazy (val, 0);
-  return 0;
 }
 
 /* Implementation of the convenience function $_isvoid.  */
diff --git a/gdb/value.h b/gdb/value.h
index 957bcd4..7ff6aa8 100644
--- a/gdb/value.h
+++ b/gdb/value.h
@@ -361,7 +361,7 @@ extern const gdb_byte *value_contents_for_printing (struct value *value);
 extern const gdb_byte *
   value_contents_for_printing_const (const struct value *value);
 
-extern int value_fetch_lazy (struct value *val);
+extern void value_fetch_lazy (struct value *val);
 
 /* If nonzero, this is the value of a variable which does not actually
    exist in the program, at least partially.  If the value is lazy,