Fix build bug in ppc-linux-nat.c
authorTom Tromey <tom@tromey.com>
Tue, 14 Feb 2023 07:28:47 +0000 (00:28 -0700)
committerTom Tromey <tom@tromey.com>
Tue, 14 Feb 2023 07:31:04 +0000 (00:31 -0700)
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.

gdb/ppc-linux-nat.c

index e4fcce694cf1da1b9a47e7f39fbe11e25cb3a324..32c0177228ad4c6d575b27977981a322d5fbfff2 100644 (file)
@@ -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)
        {