aboutsummaryrefslogtreecommitdiff
path: root/gdb/memattr.c
diff options
context:
space:
mode:
authorTom Tromey <tom@tromey.com>2021-02-26 17:25:38 -0700
committerTom Tromey <tom@tromey.com>2021-02-26 17:32:42 -0700
commit573dc0cc43f2c3ce4d28ec1aa1bf05fc43810cda (patch)
tree68d3c38bd1b2eeaacf18b2eced4fe2f7be32861f /gdb/memattr.c
parent0cf9ea0b163f4651f2d61252edc6765d98924fe8 (diff)
downloadgdb-573dc0cc43f2c3ce4d28ec1aa1bf05fc43810cda.zip
gdb-573dc0cc43f2c3ce4d28ec1aa1bf05fc43810cda.tar.gz
gdb-573dc0cc43f2c3ce4d28ec1aa1bf05fc43810cda.tar.bz2
Minor fix in skip_ctf_tests
I noticed an oddity in skip_ctf_tests -- for me it ends up caching the string "!0", because it ends with 'return ![...]'. In Tcl, this is just string concatenation. The result works because the users of this function have unbraced if conditions, like: if [skip_ctf_tests] { ... which works because "if" re-parses the returned string as an expression, and evaluates that. There's only a latent bug here, but this is also un-idiomatic, so I am checking in this patch to fix it. This way, if someone in the future uses a braced condition (which is what I normally recommend), it will continue to work. gdb/testsuite/ChangeLog 2021-02-26 Tom Tromey <tom@tromey.com> * lib/gdb.exp (skip_ctf_tests): Use expr on result.
Diffstat (limited to 'gdb/memattr.c')
0 files changed, 0 insertions, 0 deletions