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

Alexandre Julliard julliard at winehq.org
Fri Feb 11 11:16:35 CST 2011


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

Author: Austin English <austinenglish at gmail.com>
Date:   Thu Feb 10 13:33:05 2011 -0800

shlwapi/tests: Remove useless assignments (LLVM/Clang).

---

 dlls/shlwapi/tests/clist.c |   16 ++++++----------
 1 files changed, 6 insertions(+), 10 deletions(-)

diff --git a/dlls/shlwapi/tests/clist.c b/dlls/shlwapi/tests/clist.c
index 0c1704d..65d3178 100644
--- a/dlls/shlwapi/tests/clist.c
+++ b/dlls/shlwapi/tests/clist.c
@@ -371,26 +371,22 @@ static void test_CList(void)
   inserted = (LPSHLWAPI_CLIST)buff;
   inserted->ulSize = sizeof(SHLWAPI_CLIST) -1;
   inserted->ulId = 33;
-  hRet = pSHLWAPI_20(&list, inserted);
+
   /* The call succeeds but the item is not inserted, except on some early
    * versions which return failure. Wine behaves like later versions.
    */
-  if (0)
-  {
-  ok(hRet == S_OK, "failed bad element size\n");
-  }
+  pSHLWAPI_20(&list, inserted);
+
   inserted = pSHLWAPI_22(list, 33);
   ok(inserted == NULL, "inserted bad element size\n");
 
   inserted = (LPSHLWAPI_CLIST)buff;
   inserted->ulSize = 44;
   inserted->ulId = ~0U;
-  hRet = pSHLWAPI_20(&list, inserted);
+
   /* See comment above, some early versions fail this call */
-  if (0)
-  {
-  ok(hRet == S_OK, "failed adding a container\n");
-  }
+  pSHLWAPI_20(&list, inserted);
+
   item = SHLWAPI_CLIST_items;
 
   /* Look for nonexistent item in populated list */




More information about the wine-cvs mailing list