MSI: indentation and style cleanup

Alexandre Julliard julliard at wine.dyndns.org
Fri Jan 21 04:29:27 CST 2005


Mike McCormack <mike at codeweavers.com> writes:

> -    if(szFilePath) {
> +    if( szFilePath )
> +    {
>          len = MultiByteToWideChar( CP_ACP, 0, szFilePath, -1, NULL, 0 );
>          szwFilePath = HeapAlloc( GetProcessHeap(), 0, len*sizeof(WCHAR) );
> -        if( !szwFilePath)
> -            return ERROR_OUTOFMEMORY;
>          MultiByteToWideChar( CP_ACP, 0, szFilePath, -1, szwFilePath, len );
>      }

Does your coding style also forbid proper error checking, or is there
another reason for removing that check?  <g>

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list