ole32 [2/2] (msi [9.5/13]): Storage sharing should be implemented inole32, not the lower-level file API

Dmitry Timoshkov dmitry at codeweavers.com
Mon Aug 18 23:25:24 CDT 2008


"James Hawkins" <jhawkins at codeweavers.com> wrote:

> -    { FALSE, ERROR_SHARING_VIOLATION, FALSE },
> -    { TRUE, ERROR_SUCCESS, TRUE }
> +    { TRUE, ERROR_SUCCESS },
> +    { TRUE, ERROR_SUCCESS },
> +    { TRUE, ERROR_SUCCESS },
> +    { TRUE, ERROR_SUCCESS },
> +    { FALSE, ERROR_SHARING_VIOLATION },
> +    { FALSE, ERROR_SHARING_VIOLATION },
> +    { FALSE, ERROR_SHARING_VIOLATION },
> +    { TRUE, ERROR_SUCCESS },
> +    { FALSE, ERROR_SHARING_VIOLATION },
> +    { FALSE, ERROR_SHARING_VIOLATION },
> +    { FALSE, ERROR_SHARING_VIOLATION },
> +    { TRUE, ERROR_SUCCESS },
> +    { FALSE, ERROR_SHARING_VIOLATION },
> +    { FALSE, ERROR_SHARING_VIOLATION },
> +    { FALSE, ERROR_SHARING_VIOLATION },
> +    { TRUE, ERROR_SUCCESS }
>  };

You forgot to remove 'BOOL todo;' from struct access_res definition.
Also making the arrays of structures 'static const' while you are at it
won't hurt (same for other similar tests).

-- 
Dmitry.



More information about the wine-devel mailing list