Re: user[2/5]: implement A version of SPI_GETDESKWALLPAPER

[Andrey Turkin]

Andrew Ziem wrote:
> Previously, SPI_GETDESKWALLPAPER was handled by default: in the 
> switch, so the function (an A- function) would return a W string.
>
> 5e4b86d689c899fc896d2870efaca9c335f01da0
> diff --git a/dlls/user/sysparams.c b/dlls/user/sysparams.c
> index 5fed316..fb54b3b 100644
> --- a/dlls/user/sysparams.c
> +++ b/dlls/user/sysparams.c
> @@ -2496,6 +2496,16 @@ BOOL WINAPI SystemParametersInfoA( UINT 
>    break;
>      }
> - case SPI_GETDESKWALLPAPER:                  /*     115 */
> - {
> -
>    WCHAR buffer[MAX_PATH];
> -
>    ret = SystemParametersInfoW( SPI_GETDESKWALLPAPER, uiParam, buffer, fuWinIni );
> -
>    if (!WideCharToMultiByte(CP_ACP, 0, buffer, -1, pvParam, uiParam, NULL, NULL))
> -
>       ret = FALSE;
> -
>    else
> -
>       pvParam = buffer;
> - }
> - default:
>        ret = SystemParametersInfoW( uiAction, uiParam, pvParam, fuWinIni );
>        break;

I think break is needed after case. Also, for what

else pvParam = buffer;

need for?