This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[PATCH 2/3] Remove some GDB specific stuff from selftest.c
- From: Yao Qi <qiyaoltc at gmail dot com>
- To: gdb-patches at sourceware dot org
- Date: Fri, 11 Aug 2017 16:30:07 +0100
- Subject: [PATCH 2/3] Remove some GDB specific stuff from selftest.c
- Authentication-results: sourceware.org; auth=none
- References: <1502465408-24668-1-git-send-email-yao.qi@linaro.org>
The next patch moves selftest.c to common/selftest.c, so that GDBserver
can use it as well. However selftest.c uses something isn't "portable" on
GDB and GDBserver.
First, this patch removes QUIT. I don't expect that we type ctrl-c during
self/unit tests, and each test shouldn't take long time. Secondly, I
replace exception_fprintf and printf_filtered with debug_printf. Verified
that unit tests still catch fails.
gdb:
2017-08-09 Yao Qi <yao.qi@linaro.org>
* selftest.c (run_tests): Don't call QUIT. Call debug_printf
instead of exception_fprintf and printf_filtered.
---
gdb/selftest.c | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/gdb/selftest.c b/gdb/selftest.c
index 31d16af..eb7728b 100644
--- a/gdb/selftest.c
+++ b/gdb/selftest.c
@@ -44,8 +44,6 @@ run_tests (void)
for (int i = 0; i < tests.size (); ++i)
{
- QUIT;
-
TRY
{
tests[i] ();
@@ -53,7 +51,7 @@ run_tests (void)
CATCH (ex, RETURN_MASK_ERROR)
{
++failed;
- exception_fprintf (gdb_stderr, ex, _("Self test failed: "));
+ debug_printf ("Self test failed: %s\n", ex.message);
}
END_CATCH
@@ -62,7 +60,7 @@ run_tests (void)
reinit_frame_cache ();
}
- printf_filtered (_("Ran %lu unit tests, %d failed\n"),
- (long) tests.size (), failed);
+ debug_printf ("Ran %lu unit tests, %d failed\n",
+ (long) tests.size (), failed);
}
} // namespace selftests
--
1.9.1