Remove the '\n' hack in winetest_ok()

Francois Gouget fgouget at free.fr
Tue Feb 10 11:32:18 CST 2004


Now that all ok() calls have a trailing '\n', we no longer need the
winetest_ok() hack that adds a '\n' if there is none. Removing this hack
is actually necessary if we don't want ok() calls with a missing '\n' to
be introduced again.


Changelog:

 * include/wine/test.h

   Remove the winetest_ok() hack that adds a trailing '\n' if it is
missing.


Index: include/wine/test.h
===================================================================
RCS file: /home/cvs/wine/include/wine/test.h,v
retrieving revision 1.12
diff -u -r1.12 test.h
--- include/wine/test.h	11 Sep 2003 01:07:19 -0000	1.12
+++ include/wine/test.h	9 Feb 2004 22:12:51 -0000
@@ -156,7 +156,6 @@
 int winetest_ok( int condition, const char *msg, ... )
 {
     va_list valist;
-    int len;
     tls_data* data=get_tls_data();

     if (data->todo_level)
@@ -172,9 +171,6 @@
                 vfprintf(stdout, msg, valist);
                 va_end(valist);
             }
-            len=strlen(msg);
-            if (len==0 || msg[len-1]!='\n')
-                fputc( '\n', stdout );
             InterlockedIncrement(&todo_failures);
             return 0;
         }
@@ -193,9 +189,6 @@
                 vfprintf(stdout, msg, valist);
                 va_end(valist);
             }
-            len=strlen(msg);
-            if (len==0 || msg[len-1]!='\n')
-                fputc( '\n', stdout );
             InterlockedIncrement(&failures);
             return 0;
         }



-- 
Francois Gouget         fgouget at free.fr        http://fgouget.free.fr/
              Linux: Because rebooting is for adding new hardware



More information about the wine-patches mailing list