IShellFolder::CompareIDs - return only -1/0/1 in lower 16 bit.

dlls/shell32/shlfolder.c

@@ -418,7 +418,7 @@ HRESULT SHELL32_CompareIDs (IShellFolder * iface, LPARAM lParam, LPCITEMIDLIST p
       type2;
     char szTemp1[MAX_PATH];
     char szTemp2[MAX_PATH];
-    int nReturn = 0;
+    HRESULT nReturn;
     LPITEMIDLIST firstpidl,
       nextpidl1,
       nextpidl2;
@@ -429,24 +429,28 @@ HRESULT SHELL32_CompareIDs (IShellFolder * iface, LPARAM lParam, LPCITEMIDLIST p
     BOOL isEmpty2 = _ILIsDesktop (pidl2);
 
     if (isEmpty1 && isEmpty2)
-	return 0;
+        return MAKE_HRESULT( SEVERITY_SUCCESS, 0, 0 );
     if (isEmpty1)
-	return -1;
+        return MAKE_HRESULT( SEVERITY_SUCCESS, 0, (WORD)-1 );
     if (isEmpty2)
-	return 1;
+        return MAKE_HRESULT( SEVERITY_SUCCESS, 0, 1 );
 
     /* test for different types. Sort order is the PT_* constant */
     type1 = _ILGetDataPointer (pidl1)->type;
     type2 = _ILGetDataPointer (pidl2)->type;
-    if (type1 != type2)
-	return (type1 - type2);
+    if (type1 < type2)
+        return MAKE_HRESULT( SEVERITY_SUCCESS, 0, (WORD)-1 );
+    else if (type1 > type2)
+        return MAKE_HRESULT( SEVERITY_SUCCESS, 0, 1 );
 
     /* test for name of pidl */
     _ILSimpleGetText (pidl1, szTemp1, MAX_PATH);
     _ILSimpleGetText (pidl2, szTemp2, MAX_PATH);
     nReturn = strcasecmp (szTemp1, szTemp2);
-    if (nReturn != 0)
-	return nReturn;
+    if (nReturn < 0)
+        return MAKE_HRESULT( SEVERITY_SUCCESS, 0, (WORD)-1 );
+    else if (nReturn > 0)
+        return MAKE_HRESULT( SEVERITY_SUCCESS, 0, 1 );
 
     /* test of complex pidls */
     firstpidl = ILCloneFirst (pidl1);
@@ -459,11 +463,11 @@ HRESULT SHELL32_CompareIDs (IShellFolder * iface, LPARAM lParam, LPCITEMIDLIST p
     isEmpty2 = _ILIsDesktop (nextpidl2);
 
     if (isEmpty1 && isEmpty2) {
-	nReturn = 0;
+        return MAKE_HRESULT( SEVERITY_SUCCESS, 0, 0 );
     } else if (isEmpty1) {
-	nReturn = -1;
+        return MAKE_HRESULT( SEVERITY_SUCCESS, 0, (WORD)-1 );
     } else if (isEmpty2) {
-    	nReturn = 1;
+        return MAKE_HRESULT( SEVERITY_SUCCESS, 0, 1 );
     /* optimizing end */
     } else if (SUCCEEDED (IShellFolder_BindToObject (iface, firstpidl, NULL, &IID_IShellFolder, (LPVOID *) & psf))) {
 	nReturn = IShellFolder_CompareIDs (psf, lParam, nextpidl1, nextpidl2);

dlls/shell32/tests/.cvsignore

@@ -1,5 +1,6 @@
 Makefile
 generated.ok
 shlfileop.ok
+shlfolder.ok
 string.ok
 testlist.c

dlls/shell32/tests/Makefile.in

@@ -4,10 +4,12 @@ SRCDIR    = @srcdir@
 VPATH     = @srcdir@
 TESTDLL   = shell32.dll
 IMPORTS   = shell32 ole32
+EXTRALIBS = -luuid
 
 CTESTS = \
 	generated.c \
 	shlfileop.c \
+	shlfolder.c \
 	string.c
 
 @MAKE_TEST_RULES@

dlls/shell32/tests/shlfolder.c

@@ -0,0 +1,151 @@
+/*
+ * Unit test of the IShellFolder functions.
+ *
+ * Copyright 2004 Vitaliy Margolen
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+ */
+
+#include <stdarg.h>
+#include <stdio.h>
+
+#include "windef.h"
+#include "winbase.h"
+#include "wtypes.h"
+#include "shellapi.h"
+
+
+#include "shlguid.h"
+#include "shlobj.h"
+#include "shobjidl.h"
+
+
+#include "wine/unicode.h"
+#include "wine/test.h"
+#include "wine/debug.h"
+
+
+IMalloc *ppM;
+
+/* creates a file with the specified name for tests */
+void CreateTestFile(CHAR *name)
+{
+    HANDLE file;
+    DWORD written;
+
+    file = CreateFileA(name, GENERIC_WRITE, 0, NULL, CREATE_ALWAYS, 0, NULL);
+    if (file != INVALID_HANDLE_VALUE)
+    {
+	WriteFile(file, name, strlen(name), &written, NULL);
+	WriteFile(file, "\n", strlen("\n"), &written, NULL);
+	CloseHandle(file);
+    }
+}
+
+
+/* initializes the tests */
+void CreateFilesFolders(void)
+{
+    CreateDirectoryA(".\\testdir", NULL);
+    CreateDirectoryA(".\\testdir\\test.txt", NULL);
+    CreateTestFile  (".\\testdir\\test1.txt ");
+    CreateTestFile  (".\\testdir\\test2.txt ");
+    CreateTestFile  (".\\testdir\\test3.txt ");
+    CreateDirectoryA(".\\testdir\\testdir2 ", NULL);
+}
+
+/* cleans after tests */
+void Cleanup(void)
+{
+    DeleteFileA(".\\testdir\\test1.txt");
+    DeleteFileA(".\\testdir\\test2.txt");
+    DeleteFileA(".\\testdir\\test3.txt");
+    RemoveDirectoryA(".\\testdir\\test.txt");
+    RemoveDirectoryA(".\\testdir\\testdir2");
+    RemoveDirectoryA(".\\testdir");
+}
+
+
+/* perform test */
+void test_EnumObjects(IShellFolder *iFolder)
+{
+    IEnumIDList *iEnumList;
+    ITEMIDLIST *newPIDL, *(idlArr [5]);
+    ULONG NumPIDLs;
+    int i=0, j;
+    HRESULT nResult;
+
+    static const WORD iResults [5][5] =
+    {
+	{ 0,-1,-1,-1,-1},
+	{ 1, 0,-1,-1, 1},
+	{ 1, 1, 0,-1, 1},
+	{ 1, 1, 1, 0, 1},
+	{ 1,-1,-1,-1, 0}
+    };
+
+    if SUCCEEDED(IShellFolder_EnumObjects(iFolder, NULL, SHCONTF_FOLDERS | SHCONTF_NONFOLDERS | SHCONTF_INCLUDEHIDDEN, &iEnumList))
+    {
+	while (IEnumIDList_Next(iEnumList, 1, &newPIDL, &NumPIDLs) == S_OK)
+	{
+	    idlArr[i++] = newPIDL;
+	}
+	/* This fails on windows */
+	/* IEnumIDList_Release(iEnumList); */
+    
+	for (i=0;i<5;i++) for (j=0;j<5;j++)
+	{
+	    nResult = IShellFolder_CompareIDs(iFolder, 0, idlArr[i], idlArr[j]);
+	    ok(nResult == iResults[i][j], "Got %lx expected %x\n", nResult, iResults[i][j]);
+	}
+
+	for (i=0;i<5;i++)
+	    IMalloc_Free(ppM, idlArr[i]);
+    }
+}
+
+START_TEST(shlfolder)
+{
+    ITEMIDLIST *newPIDL;
+    IShellFolder *IDesktopFolder, *testIShellFolder;
+    WCHAR cCurrDirW [MAX_PATH];
+    static const WCHAR cTestDirW[] = {'\\','t','e','s','t','d','i','r',0};
+
+    
+    GetCurrentDirectoryW(MAX_PATH, cCurrDirW);
+    strcatW(cCurrDirW, cTestDirW);
+
+    
+    if(!SUCCEEDED(SHGetMalloc(&ppM)))
+	return;
+
+    CreateFilesFolders();
+    SHGetDesktopFolder(&IDesktopFolder);
+
+    if (SUCCEEDED(IShellFolder_ParseDisplayName(IDesktopFolder, NULL, NULL, cCurrDirW, NULL, &newPIDL, 0)))
+    {
+	if (SUCCEEDED(IShellFolder_BindToObject(IDesktopFolder, newPIDL, NULL, (REFIID)&IID_IShellFolder, (LPVOID *)&testIShellFolder)))
+	{
+	    test_EnumObjects(testIShellFolder);
+	    
+	    /* This fails on windows */
+	    /* IShellFolder_Release(newIShellFolder); */
+
+	    IMalloc_Free(ppM, newPIDL);
+	}
+    }
+
+    Cleanup();
+}