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?