On 13/02/2024 22:01, Thiago Jung Bauermann wrote:
Hello again,

Thiago Jung Bauermann <thiago.bauerm...@linaro.org> writes:

diff --git a/gdb/testsuite/gdb.threads/threadcrash.exp 
b/gdb/testsuite/gdb.threads/threadcrash.exp
index 3905ad6f9362..944fbcac1b18 100644
--- a/gdb/testsuite/gdb.threads/threadcrash.exp
+++ b/gdb/testsuite/gdb.threads/threadcrash.exp
@@ -28,7 +28,7 @@ proc test_thread_count {} {
      set thread_count 0
gdb_test_multiple "print \$_inferior_thread_count" "getting thread count" {
-       -re ".* = (\[0-9]+).*" {
+       -re ".* = (\[0-9]+)\r\n$::gdb_prompt " {
            set thread_count $expect_out(1,string)
            gdb_assert {$thread_count == 7}
        }
It occurred to me that this gdb_test_multiple call is very similar to
the function get_valueof in lib/gdb.exp, so one suggestion is to use
that function instead. It uses "-re -wrap" so it expects the GDB prompt.

This makes a lot of sense. thanks for the help! And for making this CI, it is clearly making a difference! :D

--
Cheers,
Guinevere Larsen
She/Her/Hers

_______________________________________________
linaro-toolchain mailing list -- linaro-toolchain@lists.linaro.org
To unsubscribe send an email to linaro-toolchain-le...@lists.linaro.org

Reply via email to