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 --git a/gdb/ChangeLog b/gdb/ChangeLog
index 463665317e10899d1a93ea1d1bf61a5a68c36516..29b7da7fe1144d590ae4afcd418eb6ffcf59b189 100644 (file)
--- 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 86d7d0029d2d680887e4b06cda7b3763b4362b5e..a01f3902741a3f361b381126041497db51ff8db4 100644 (file)
--- 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 957bcd494636e6abf318a6d50341d92a5005de75..7ff6aa8fd98741e3c5a480250ea80ea4ea7644e5 100644 (file)
--- 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,