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.


Reply via email to