include: Create todo_wine_if.
Bernhard Übelacker
bernhardu at vr-web.de
Fri Jan 22 10:50:08 CST 2016
Currently list based tests, where just some elements are todo_wine, need
the same ok()-line twice, one without and one with todo_wine.
(e.g. dlls/msvcr120/tests/msvcr120.c line 243)
This patch makes it possible to use a single ok()-line
prefixed with a todo_wine_if(condition).
e.g.:
todo_wine_if(tests[i].todo)
ok(...);
Would such a change be acceptable?
---
include/wine/test.h | 24 ++++++++++++++----------
1 file changed, 14 insertions(+), 10 deletions(-)
diff --git a/include/wine/test.h b/include/wine/test.h
index bed44f9..2fc1322 100644
--- a/include/wine/test.h
+++ b/include/wine/test.h
@@ -56,9 +56,11 @@ extern int winetest_interactive;
extern const char *winetest_platform;
extern void winetest_set_location( const char* file, int line );
-extern void winetest_start_todo( const char* platform );
+extern void winetest_start_todo_if( const int is_todo, const char* platform );
+#define winetest_start_todo(platform) winetest_start_todo_if(TRUE, (platform))
extern int winetest_loop_todo(void);
-extern void winetest_end_todo( const char* platform );
+extern void winetest_end_todo_if( const int is_todo, const char* platform );
+#define winetest_end_todo(platform) winetest_end_todo_if(TRUE, (platform))
extern int winetest_get_mainargs( char*** pargv );
extern LONG winetest_get_failures(void);
extern void winetest_add_failures( LONG new_failures );
@@ -118,10 +120,12 @@ extern void __winetest_cdecl winetest_trace( const char *msg, ... ) WINETEST_PRI
#define win_skip win_skip_(__FILE__, __LINE__)
#define trace trace_(__FILE__, __LINE__)
-#define todo(platform) for (winetest_start_todo(platform); \
- winetest_loop_todo(); \
- winetest_end_todo(platform))
-#define todo_wine todo("wine")
+#define todo_if(is_todo, platform) for (winetest_start_todo_if(is_todo, platform); \
+ winetest_loop_todo(); \
+ winetest_end_todo_if(is_todo, platform))
+#define todo_wine todo_if(TRUE, "wine")
+#define todo_wine_if(is_todo) todo_if((is_todo), "wine")
+
#ifdef NONAMELESSUNION
@@ -395,10 +399,10 @@ void __winetest_cdecl winetest_win_skip( const char *msg, ... )
__winetest_va_end(valist);
}
-void winetest_start_todo( const char* platform )
+void winetest_start_todo_if( const int is_todo, const char* platform )
{
tls_data* data=get_tls_data();
- if (strcmp(winetest_platform,platform)==0)
+ if (strcmp(winetest_platform,platform)==0 && is_todo)
data->todo_level++;
data->todo_do_loop=1;
}
@@ -411,9 +415,9 @@ int winetest_loop_todo(void)
return do_loop;
}
-void winetest_end_todo( const char* platform )
+void winetest_end_todo_if( const int is_todo, const char* platform )
{
- if (strcmp(winetest_platform,platform)==0)
+ if (strcmp(winetest_platform,platform)==0 && is_todo)
{
tls_data* data=get_tls_data();
data->todo_level--;
--
2.1.4
More information about the wine-patches
mailing list