diff options
author | Simon Marchi <simon.marchi@ericsson.com> | 2015-07-06 13:04:11 -0400 |
---|---|---|
committer | Simon Marchi <simon.marchi@ericsson.com> | 2015-07-06 13:04:11 -0400 |
commit | a844296a982bdff7c14dab355de0fca75486add6 (patch) | |
tree | a195704067eff26020a86fbeb30bcc7cffb6762e /gdb | |
parent | f41cbf58f4af65e8612585c45208f3690a39ea28 (diff) | |
download | binutils-a844296a982bdff7c14dab355de0fca75486add6.zip binutils-a844296a982bdff7c14dab355de0fca75486add6.tar.gz binutils-a844296a982bdff7c14dab355de0fca75486add6.tar.bz2 |
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.
Diffstat (limited to 'gdb')
-rw-r--r-- | gdb/ChangeLog | 6 | ||||
-rw-r--r-- | gdb/value.c | 17 | ||||
-rw-r--r-- | 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, |