shell32 patch 21

Martin Fuchs martin-fuchs at gmx.net
Wed Jan 28 14:29:17 CST 2004


Changelog:
- remove unneccesary ISF_MyComputer_Constructor() declaration; it's already there in shell32_main.h
- improve returned error code of ISF_MyComputer_fnParseDisplayName()
- remove wrong comment from ISF_MyComputer_fnGetDisplayNameOf()


Index: shfldr_mycomp.c
===================================================================
RCS file: /home/wine/wine/dlls/shell32/shfldr_mycomp.c,v
retrieving revision 1.14
diff -u -p -d -w -b -r1.14 shfldr_mycomp.c
--- shfldr_mycomp.c	23 Jan 2004 04:25:26 -0000	1.14
+++ shfldr_mycomp.c	28 Jan 2004 20:25:57 -0000
@@ -67,8 +67,6 @@ typedef struct {
 static struct ICOM_VTABLE (IShellFolder2) vt_ShellFolder2;
 static struct ICOM_VTABLE (IPersistFolder2) vt_PersistFolder2;
 
-HRESULT WINAPI ISF_MyComputer_Constructor (IUnknown * pUnkOuter, REFIID riid, LPVOID * ppv);
-
 #define _IPersistFolder2_Offset ((int)(&(((IGenericSFImpl*)0)->lpVtblPersistFolder2)))
 #define _ICOM_THIS_From_IPersistFolder2(class, name) class* This = (class*)(((char*)name)-_IPersistFolder2_Offset);
 
@@ -195,7 +193,7 @@ ISF_MyComputer_fnParseDisplayName (IShel
 {
     ICOM_THIS (IGenericSFImpl, iface);
 
-    HRESULT hr = E_OUTOFMEMORY;
+    HRESULT hr = E_INVALIDARG;
     LPCWSTR szNext = NULL;
     WCHAR szElement[MAX_PATH];
     CHAR szTempA[MAX_PATH];
@@ -418,7 +416,7 @@ ISF_MyComputer_fnGetUIObjectOf (IShellFo
 	    hr = E_NOINTERFACE;
 	}
 
-	if (!pObj)
+	if (SUCCEEDED(hr) && !pObj)
 	    hr = E_OUTOFMEMORY;
 
 	*ppvOut = pObj;
@@ -429,10 +427,6 @@ ISF_MyComputer_fnGetUIObjectOf (IShellFo
 
 /**************************************************************************
 *	ISF_MyComputer_fnGetDisplayNameOf
-*
-* NOTES
-*	The desktopfolder creates only complete paths (SHGDN_FORPARSING).
-*	SHGDN_INFOLDER makes no sense.
 */
 static HRESULT WINAPI ISF_MyComputer_fnGetDisplayNameOf (IShellFolder2 * iface, LPCITEMIDLIST pidl, DWORD dwFlags, LPSTRRET strRet)
 {






More information about the wine-patches mailing list