Austin English : avifil32/tests: Remove useless variable assignments (LLVM/ Clang).

Alexandre Julliard julliard at winehq.org
Mon Feb 7 16:06:38 CST 2011


Module: wine
Branch: master
Commit: 0d258b47371d9ca0b8dfd9c7d7222c4236e642f0
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=0d258b47371d9ca0b8dfd9c7d7222c4236e642f0

Author: Austin English <austinenglish at gmail.com>
Date:   Sat Feb  5 20:56:17 2011 -0800

avifil32/tests: Remove useless variable assignments (LLVM/Clang).

---

 dlls/avifil32/tests/api.c |   17 ++++++++---------
 1 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/dlls/avifil32/tests/api.c b/dlls/avifil32/tests/api.c
index 14ad4b4..4b87806 100644
--- a/dlls/avifil32/tests/api.c
+++ b/dlls/avifil32/tests/api.c
@@ -232,11 +232,7 @@ static void test_EditStreamSetInfo(void)
     hres = CreateEditableStream(&stream, NULL);
     ok(hres == AVIERR_OK, "got 0x%08X, expected AVIERR_OK\n", hres);
 
-
-    if(0)    /* Crashing - first parameter not checked */
-        hres = EditStreamSetInfo(NULL, &info, sizeof(AVISTREAMINFO) );
-
-             /* Size parameter is somehow checked (notice the crash with size=-1 below) */
+    /* Size parameter is somehow checked (notice the crash with size=-1 below) */
     hres = EditStreamSetInfo(stream, NULL, 0);
     ok( hres == AVIERR_BADSIZE, "got 0x%08X, expected AVIERR_BADSIZE\n", hres);
 
@@ -244,11 +240,14 @@ static void test_EditStreamSetInfo(void)
     ok( hres == AVIERR_BADSIZE, "got 0x%08X, expected AVIERR_BADSIZE\n", hres);
 
     if(0)
-    {        /* Crashing - second parameter not checked */
-        hres = EditStreamSetInfo(stream, NULL, sizeof(AVISTREAMINFO) );
+    {   
+        /* Crashing - first parameter not checked */
+        EditStreamSetInfo(NULL, &info, sizeof(AVISTREAMINFO) );
+
+        /* Crashing - second parameter not checked */
+        EditStreamSetInfo(stream, NULL, sizeof(AVISTREAMINFO) );
 
-        hres = EditStreamSetInfo(stream, NULL, -1);
-        ok( hres == AVIERR_BADSIZE, "got 0x%08X, expected AVIERR_BADSIZE\n", hres);
+        EditStreamSetInfo(stream, NULL, -1);
     }
 
     hres = AVIStreamInfo(stream, &info, sizeof(AVISTREAMINFO) );




More information about the wine-cvs mailing list