diff options
author | Tom Tromey <tom@tromey.com> | 2021-02-26 17:25:38 -0700 |
---|---|---|
committer | Tom Tromey <tom@tromey.com> | 2021-02-26 17:32:42 -0700 |
commit | 573dc0cc43f2c3ce4d28ec1aa1bf05fc43810cda (patch) | |
tree | 68d3c38bd1b2eeaacf18b2eced4fe2f7be32861f /gdb/exc_request.defs | |
parent | 0cf9ea0b163f4651f2d61252edc6765d98924fe8 (diff) | |
download | gdb-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/exc_request.defs')
0 files changed, 0 insertions, 0 deletions