Re: shellpath.c: Avoid copying invalid data on error

[Alexandre Julliard]

Juan Lang  writes:
> The following is a better patch, I think, in that
> - it doesn't add a break statement to the default case
>
> (some compilers warn on this)

I seriously doubt that, what compiler does this for you?  It's much
better practice to always add break to all cases, otherwise when
someone adds some code at the end of the switch they will forget to
add a break to the previous case.
> - WCHAR      szBuildPath[MAX_PATH], szTemp[MAX_PATH];
> - WCHAR      szBuildPath[MAX_PATH], szTemp[MAX_PATH] = { 0 };

This fills the whole buffer with zeroes, that's not really needed.