diff options
| author | Tom Tromey <tromey@adacore.com> | 2025-05-12 11:30:33 -0600 |
|---|---|---|
| committer | Tom Tromey <tromey@adacore.com> | 2025-05-14 09:18:01 -0600 |
| commit | f8f02611616a7dfd556d0052001c9845852b57b0 (patch) | |
| tree | d7afe29ac2aad17593210215bd2625b53ed9047a | |
| parent | 9f400a2f15c4892537727f75acb5e3b463aa0ea3 (diff) | |
| download | binutils-f8f02611616a7dfd556d0052001c9845852b57b0.zip binutils-f8f02611616a7dfd556d0052001c9845852b57b0.tar.gz binutils-f8f02611616a7dfd556d0052001c9845852b57b0.tar.bz2 | |
Fix create_breakpoint_parse_arg_string self-test
The emoji patch broke the create_breakpoint_parse_arg_string self-test
when gdb is running on a suitable terminal. The problem is that the
test case doesn't take the error prefix string into account.
This patch fixes the test by having it compare the exception message
directly, rather than relying on the result of exception_print. I did
try a different approach, of having the test mimic exception_print,
but this one seemed cleaner to me.
Approved-By: Simon Marchi <simon.marchi@efficios.com>
| -rw-r--r-- | gdb/break-cond-parse.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/gdb/break-cond-parse.c b/gdb/break-cond-parse.c index 705f4f6..caf13a7 100644 --- a/gdb/break-cond-parse.c +++ b/gdb/break-cond-parse.c @@ -569,10 +569,10 @@ test (const char *input, const char *condition, int thread = -1, gdb::unique_xmalloc_ptr<char> extracted_rest; int extracted_thread, extracted_inferior, extracted_task; bool extracted_force_condition; - std::string exception_msg, error_str; + std::string exception_msg; - if (error_msg != nullptr) - error_str = std::string (error_msg) + "\n"; + if (error_msg == nullptr) + error_msg = ""; try { @@ -584,10 +584,7 @@ test (const char *input, const char *condition, int thread = -1, } catch (const gdb_exception_error &ex) { - string_file buf; - - exception_print (&buf, ex); - exception_msg = buf.release (); + exception_msg = ex.what (); } if ((condition == nullptr) != (extracted_condition.get () == nullptr) @@ -599,7 +596,7 @@ test (const char *input, const char *condition, int thread = -1, || inferior != extracted_inferior || task != extracted_task || force != extracted_force_condition - || exception_msg != error_str) + || exception_msg != error_msg) { if (run_verbose ()) { |