Re: dlls/shell32/pidl.c -- adjust to unsignedness of a type

2008-02-17 Thread Gerald Pfeifer
heck in ILGetDisplayNameExW(). Index: pidl.c ======= RCS file: /home/wine/wine/dlls/shell32/pidl.c,v retrieving revision 1.159 diff -u -3 -p -r1.159 pidl.c --- pidl.c 16 Feb 2008 15:58:53 - 1.159 +++ pidl.c 17 Feb 2008 18:44:54 - @@ -97,7 +97,7 @@ BOOL WINAPI ILGetDisplayNa

Re: dlls/shell32/pidl.c -- adjust to unsignedness of a type

2008-02-16 Thread Alexandre Julliard
Gerald Pfeifer <[EMAIL PROTECTED]> writes: > @@ -109,7 +109,7 @@ BOOL WINAPI ILGetDisplayNameExW(LPSHELLF > return FALSE; > } > > -if (type >= 0 && type <= 2) > +if (type <= 2) > { > switch (type) > { The test should probably be removed completel

RE: dlls/shell32/pidl.c

2005-08-15 Thread Ge van Geldorp
> From: Michael Jung > > SHBindToParent does not allocate a > new PIDL for pidlLast, but returns a pointer to right > location in pidl. This means you should not free it. > There's still the problem that the shell folder isn't > released in failure cases. > > Sorry, I didn't realize this. I

Re: dlls/shell32/pidl.c

2005-08-15 Thread Michael Jung
Hi Ge, On Saturday 13 August 2005 20:55, Ge van Geldorp wrote: > dwAttributes = SFGAO_FILESYSTEM; > hr = IShellFolder_GetAttributesOf(psfFolder, 1, &pidlLast, > &dwAttributes); -if (FAILED(hr) || !(dwAttributes & SFGAO_FILESYSTEM)) > return FALSE; +if (FAILED(hr) || !(dwAttribute