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>
This commit is contained in:
Daniel Vetter 2013-08-12 11:57:32 +02:00
parent d65d19d4f4
commit 16024841a1
2 changed files with 3 additions and 3 deletions

View File

@ -717,7 +717,7 @@ out:
* outside of places protected by drmtest_run_subtest checks - the piglit * outside of places protected by drmtest_run_subtest checks - the piglit
* runner adds every line to the subtest list. * 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); assert(in_subtest == false);

View File

@ -98,8 +98,8 @@ void drmtest_progress(const char *header, uint64_t i, uint64_t total);
/* subtest infrastructure */ /* subtest infrastructure */
jmp_buf drmtest_subtest_jmpbuf; jmp_buf drmtest_subtest_jmpbuf;
void drmtest_subtest_init(int argc, char **argv); void drmtest_subtest_init(int argc, char **argv);
bool drmtest_run_subtest(const char *subtest_name); bool __drmtest_run_subtest(const char *subtest_name);
#define drmtest_subtest_block(name) for (; drmtest_run_subtest((name)) && \ #define drmtest_subtest_block(name) for (; __drmtest_run_subtest((name)) && \
(setjmp(drmtest_subtest_jmpbuf) == 0); \ (setjmp(drmtest_subtest_jmpbuf) == 0); \
drmtest_success()) drmtest_success())
bool drmtest_only_list_subtests(void); bool drmtest_only_list_subtests(void);