Joerg Mayer
Tue, 02 Jul 2002 13:05:04 -0700
On Tue, Jul 02, 2002 at 03:34:44PM -0400, Gregg Mattinson wrote:
> - void functions can't return a value, even if it is the return value from
> another void function.
Just looking at the patch a bit. This looks wrong to me. If you want to
make the changes, you should convert return xxx() into { xxx(); return }
> void WINAPI PathStripPathAW(LPVOID lpszPath)
> {
> if (SHELL_OsIsUnicode())
> - return PathStripPathW(lpszPath);
> - return PathStripPathA(lpszPath);
> + PathStripPathW(lpszPath);
> + PathStripPathA(lpszPath); <----- should not be executed if condition is
>met
> }
This pattern is repeated several times.
Ciao
Jörg
--
Joerg Mayer <[EMAIL PROTECTED]>
I found out that "pro" means "instead of" (as in proconsul). Now I know
what proactive means.