aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Tromey <tom@tromey.com>2023-02-14 00:28:47 -0700
committerTom Tromey <tom@tromey.com>2023-02-14 00:31:04 -0700
commitf54cd6441de5c057b43ea1501c1b075984067e7c (patch)
treee82abe213234310f396c379bd7ce9580510a76a3
parent978042640c7da4ee0d14c9c6e20fe7b6c07615fb (diff)
downloadgdb-f54cd6441de5c057b43ea1501c1b075984067e7c.zip
gdb-f54cd6441de5c057b43ea1501c1b075984067e7c.tar.gz
gdb-f54cd6441de5c057b43ea1501c1b075984067e7c.tar.bz2
Fix build bug in ppc-linux-nat.c
The buildbot pointed out that my value refactoring series introduced a bug in ppc-linux-nat.c: ../../binutils-gdb/gdb/ppc-linux-nat.c: In member function β€˜int ppc_linux_nat_target::num_memory_accesses(const std::vector<gdb::ref_ptr<value, value_ref_policy> >&)’: ../../binutils-gdb/gdb/ppc-linux-nat.c:2458:44: error: expected unqualified-id before β€˜->’ token 2458 | if (VALUE_LVAL (v) == not_lval || v->->deprecated_modifiable () == 0) I don't know how that happened, but I am checking in this patch which I think should fix it. It just removes the second "->". I can't readily test this, so perhaps there's another bug lurking after this one.
-rw-r--r--gdb/ppc-linux-nat.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/ppc-linux-nat.c b/gdb/ppc-linux-nat.c
index e4fcce6..32c0177 100644
--- a/gdb/ppc-linux-nat.c
+++ b/gdb/ppc-linux-nat.c
@@ -2455,7 +2455,7 @@ ppc_linux_nat_target::num_memory_accesses (const std::vector<value_ref_ptr>
struct value *v = iter.get ();
/* Constants and values from the history are fine. */
- if (v->lval () == not_lval || v->->deprecated_modifiable () == 0)
+ if (v->lval () == not_lval || v->deprecated_modifiable () == 0)
continue;
else if (v->lval () == lval_memory)
{