diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2013-08-12 11:57:32 +0200 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2013-08-12 11:57:32 +0200 |
| commit | 16024841a12f71d52aea12603d9fb1cb63124fb6 (patch) | |
| tree | 34e3f3ad7f91d527aeaff85b3bc6adfc23b85b24 | |
| parent | d65d19d4f4ad98af42c628f3a81d273ce6ac4e5a (diff) | |
lib/drmtest: __ prefix for drmtest_run_subtest
Now that callers must setup the longjmp target (we can't do that in
run_subtest itself since that callframe won't survive) tests shouldn't
call this function directly any more.
Make this clear by adding a __ prefix.
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
| -rw-r--r-- | lib/drmtest.c | 2 | ||||
| -rw-r--r-- | lib/drmtest.h | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/lib/drmtest.c b/lib/drmtest.c index 7e3c1d81c..5f1b4ae95 100644 --- a/lib/drmtest.c +++ b/lib/drmtest.c @@ -717,7 +717,7 @@ out: * outside of places protected by drmtest_run_subtest checks - the piglit * runner adds every line to the subtest list. */ -bool drmtest_run_subtest(const char *subtest_name) +bool __drmtest_run_subtest(const char *subtest_name) { assert(in_subtest == false); diff --git a/lib/drmtest.h b/lib/drmtest.h index b6874117e..0182e09cd 100644 --- a/lib/drmtest.h +++ b/lib/drmtest.h @@ -98,8 +98,8 @@ void drmtest_progress(const char *header, uint64_t i, uint64_t total); /* subtest infrastructure */ jmp_buf drmtest_subtest_jmpbuf; void drmtest_subtest_init(int argc, char **argv); -bool drmtest_run_subtest(const char *subtest_name); -#define drmtest_subtest_block(name) for (; drmtest_run_subtest((name)) && \ +bool __drmtest_run_subtest(const char *subtest_name); +#define drmtest_subtest_block(name) for (; __drmtest_run_subtest((name)) && \ (setjmp(drmtest_subtest_jmpbuf) == 0); \ drmtest_success()) bool drmtest_only_list_subtests(void); |