[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: [PATCH] shlwapi: Handle PathCanonicalize with too long paths
From:       Robert Shearman <rob () codeweavers ! com>
Date:       2008-02-29 11:09:46
Message-ID: 47C7E7FA.3000307 () codeweavers ! com
[Download RAW message or body]

Marcus Meissner wrote:
> lpszPath)
>    {
>      WCHAR szPath[MAX_PATH];
>      WCHAR szBuff[MAX_PATH];
> -    MultiByteToWideChar(CP_ACP,0,lpszPath,-1,szPath,MAX_PATH);
> +    DWORD le = GetLastError();
> +    int ret = MultiByteToWideChar(CP_ACP,0,lpszPath,-1,szPath,MAX_PATH);
> +
> +    if (!ret) {
> +	FIXME("Failed to convert string to widechar (too long?), LE %d.\n", GetLastError());
> +	SetLastError (le); /* failure does not change Lasterror, see testcase. */
> +	return FALSE;
> +    }
>   

Hi Marcus,

FIXME should be used for unimplemented functionality, not for 
highlighting application bugs that will happen on Windows too. WARN 
would be more appropriate here.

-- 
Rob Shearman



[prev in list] [next in list] [prev in thread] [next in thread]