Austin English : explorerframe/tests: Don' t assign values to variables that are never checked (LLVM/Clang).
Module: wine Branch: master Commit: 5f3ff6583928217a3d6fbd7465b9a9f1652faad1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5f3ff6583928217a3d6fbd7465...
Author: Austin English austinenglish@gmail.com Date: Fri Feb 4 17:06:20 2011 -0800
explorerframe/tests: Don't assign values to variables that are never checked (LLVM/Clang).
---
dlls/explorerframe/tests/nstc.c | 42 +++++++++++++++++++------------------- 1 files changed, 21 insertions(+), 21 deletions(-)
diff --git a/dlls/explorerframe/tests/nstc.c b/dlls/explorerframe/tests/nstc.c index 047d7cd..3ea408f 100644 --- a/dlls/explorerframe/tests/nstc.c +++ b/dlls/explorerframe/tests/nstc.c @@ -1084,8 +1084,8 @@ static void test_basics(void) if(0) { /* Crashes under Windows 7 */ - hr = INameSpaceTreeControl_AppendRoot(pnstc, NULL, SHCONTF_FOLDERS, 0, NULL); - hr = INameSpaceTreeControl_InsertRoot(pnstc, 0, NULL, SHCONTF_FOLDERS, 0, NULL); + INameSpaceTreeControl_AppendRoot(pnstc, NULL, SHCONTF_FOLDERS, 0, NULL); + INameSpaceTreeControl_InsertRoot(pnstc, 0, NULL, SHCONTF_FOLDERS, 0, NULL); }
/* Note the usage of psidesktop and psidesktop2 */ @@ -1132,7 +1132,7 @@ static void test_basics(void) if(0) { /* Crashes on native. */ - hr = INameSpaceTreeControl_GetRootItems(pnstc, NULL); + INameSpaceTreeControl_GetRootItems(pnstc, NULL); }
hr = INameSpaceTreeControl_GetRootItems(pnstc, &psia); @@ -1360,7 +1360,7 @@ static void test_basics(void) if(0) { /* Crashes under Windows 7 */ - hr = INameSpaceTreeControl_GetSelectedItems(pnstc, NULL); + INameSpaceTreeControl_GetSelectedItems(pnstc, NULL); }
psia = (void*)0xdeadbeef; @@ -1550,9 +1550,9 @@ static void test_basics(void) if(0) { /* Crashes under win 7 */ - hr = INameSpaceTreeControl_GetItemRect(pnstc, NULL, NULL); - hr = INameSpaceTreeControl_GetItemRect(pnstc, psitestdir, NULL); - hr = INameSpaceTreeControl_GetItemRect(pnstc, NULL, &rc); + INameSpaceTreeControl_GetItemRect(pnstc, NULL, NULL); + INameSpaceTreeControl_GetItemRect(pnstc, psitestdir, NULL); + INameSpaceTreeControl_GetItemRect(pnstc, NULL, &rc); }
hr = INameSpaceTreeControl_GetItemRect(pnstc, psitestdir, &rc); @@ -1690,10 +1690,10 @@ static void test_basics(void) if(0) { /* Crashes under Windows 7 */ - hr = INameSpaceTreeControl_GetItemCustomState(pnstc, NULL, NULL); - hr = INameSpaceTreeControl_GetItemCustomState(pnstc, NULL, &cbstate); - hr = INameSpaceTreeControl_GetItemCustomState(pnstc, psitestdir, NULL); - hr = INameSpaceTreeControl_SetItemCustomState(pnstc, NULL, 0); + INameSpaceTreeControl_GetItemCustomState(pnstc, NULL, NULL); + INameSpaceTreeControl_GetItemCustomState(pnstc, NULL, &cbstate); + INameSpaceTreeControl_GetItemCustomState(pnstc, psitestdir, NULL); + INameSpaceTreeControl_SetItemCustomState(pnstc, NULL, 0); }
hr = INameSpaceTreeControl_AppendRoot(pnstc, psitestdir, @@ -1807,9 +1807,9 @@ static void test_events(void) if(0) { /* Crashes native */ - hr = INameSpaceTreeControl_TreeAdvise(pnstc, NULL, NULL); - hr = INameSpaceTreeControl_TreeAdvise(pnstc, NULL, &cookie1); - hr = INameSpaceTreeControl_TreeAdvise(pnstc, (IUnknown*)pnstce, NULL); + INameSpaceTreeControl_TreeAdvise(pnstc, NULL, NULL); + INameSpaceTreeControl_TreeAdvise(pnstc, NULL, &cookie1); + INameSpaceTreeControl_TreeAdvise(pnstc, (IUnknown*)pnstce, NULL); }
/* TreeAdvise in NameSpaceTreeController seems to support only one @@ -2044,12 +2044,12 @@ static void test_events(void) if(0) { /* Crashes on Windows 7 */ - hr = INameSpaceTreeControl_SetItemState(pnstc, NULL, 0, 0); - hr = INameSpaceTreeControl_GetItemState(pnstc, NULL, 0, NULL); - hr = INameSpaceTreeControl_GetItemState(pnstc, psidesktop, 0, NULL); - hr = INameSpaceTreeControl_GetItemState(pnstc, NULL, 0, &itemstate); - hr = INameSpaceTreeControl_GetItemState(pnstc, psidesktop, 0, NULL); - hr = INameSpaceTreeControl_GetItemState(pnstc, NULL, 0, &itemstate); + INameSpaceTreeControl_SetItemState(pnstc, NULL, 0, 0); + INameSpaceTreeControl_GetItemState(pnstc, NULL, 0, NULL); + INameSpaceTreeControl_GetItemState(pnstc, psidesktop, 0, NULL); + INameSpaceTreeControl_GetItemState(pnstc, NULL, 0, &itemstate); + INameSpaceTreeControl_GetItemState(pnstc, psidesktop, 0, NULL); + INameSpaceTreeControl_GetItemState(pnstc, NULL, 0, &itemstate); }
itemstate = 0xDEADBEEF; @@ -2302,7 +2302,7 @@ static void test_events(void) if(0) { /* Crashes on Windows 7 */ - hr = INameSpaceTreeControl_EnsureItemVisible(pnstc, NULL); + INameSpaceTreeControl_EnsureItemVisible(pnstc, NULL); }
hr = INameSpaceTreeControl_EnsureItemVisible(pnstc, psidesktop);
-
Alexandre Julliard