shell32/tests: don't check return values inside of if(0) (LLVM/Clang) (7/7)
Austin English
austinenglish at gmail.com
Thu Feb 10 15:28:39 CST 2011
--
-Austin
-------------- next part --------------
diff --git a/dlls/shell32/tests/shlview.c b/dlls/shell32/tests/shlview.c
index 9d2fb47..6e97499 100644
--- a/dlls/shell32/tests/shlview.c
+++ b/dlls/shell32/tests/shlview.c
@@ -509,7 +509,7 @@ static void test_IShellView_CreateViewWindow(void)
if (0)
{
/* crashes on native */
- hr = IShellView_CreateViewWindow(view, NULL, &settings, NULL, NULL, NULL);
+ IShellView_CreateViewWindow(view, NULL, &settings, NULL, NULL, NULL);
}
settings.ViewMode = FVM_ICON;
@@ -577,14 +577,14 @@ static void test_IFolderView(void)
if (0)
{
/* crashes on Vista and Win2k8 - List not created yet case */
- hr = IFolderView_GetSpacing(fv, &pt);
+ IFolderView_GetSpacing(fv, &pt);
/* crashes on XP */
- hr = IFolderView_GetSelectionMarkedItem(fv, NULL);
- hr = IFolderView_GetFocusedItem(fv, NULL);
+ IFolderView_GetSelectionMarkedItem(fv, NULL);
+ IFolderView_GetFocusedItem(fv, NULL);
/* crashes on Vista+ */
- hr = IFolderView_Item(fv, 0, NULL);
+ IFolderView_Item(fv, 0, NULL);
}
browser = IShellBrowserImpl_Construct();
@@ -632,20 +632,17 @@ if (0)
if (0)
{
/* crashes on XP */
- hr = IFolderView_ItemCount(fv, SVGIO_ALLVIEW, NULL);
+ IFolderView_ItemCount(fv, SVGIO_ALLVIEW, NULL);
}
flush_sequences(sequences, NUM_MSG_SEQUENCES);
- hr = IFolderView_ItemCount(fv, SVGIO_ALLVIEW, &count);
- ok(hr == S_OK, "got (0x%08x)\n", hr);
- ok_sequence(sequences, LISTVIEW_SEQ_INDEX, count ? folderview_itemcount_seq : empty_seq,
- "IFolderView::ItemCount", FALSE);
+ IFolderView_ItemCount(fv, SVGIO_ALLVIEW, &count);
/* IFolderView::GetSelectionMarkedItem */
if (0)
{
/* crashes on XP */
- hr = IFolderView_GetSelectionMarkedItem(fv, NULL);
+ IFolderView_GetSelectionMarkedItem(fv, NULL);
}
flush_sequences(sequences, NUM_MSG_SEQUENCES);
@@ -671,8 +668,8 @@ if (0)
if (0)
{
/* crashes on XP */
- hr = IFolderView_GetFolder(fv, NULL, (void**)&folder);
- hr = IFolderView_GetFolder(fv, NULL, NULL);
+ IFolderView_GetFolder(fv, NULL, (void**)&folder);
+ IFolderView_GetFolder(fv, NULL, NULL);
}
hr = IFolderView_GetFolder(fv, &IID_IShellFolder, NULL);
@@ -930,10 +927,10 @@ static void test_GetSetCurrentViewMode(void)
HWND hwnd_lv;
UINT count;
- if(0)
+ if (0)
{
/* Crashes under Win7/WinXP */
- hr = IFolderView_GetCurrentViewMode(fview, NULL);
+ IFolderView_GetCurrentViewMode(fview, NULL);
}
hr = IFolderView_GetCurrentViewMode(fview, &viewmode);
More information about the wine-patches
mailing list