Re: [PATCH] include: Add IColumnsInfo interface (resend)

[Nikolay Sivov]

On 04.02.2016 12:31, Alistair Leslie-Hughes wrote:
> +[
> - object,
> - uuid(0c733a11-2a1c-11ce-ade5-00aa0044773d),
> - pointer_default(unique)
>
> +]
> +interface IColumnsInfo : IUnknown
> +{
> - [local]
> - HRESULT GetColumnInfo([in, out] DBORDINAL *columns,[out, size_is(*columns)] DBCOLUMNINFO **colinfo,
> -
>    [out] OLECHAR *stringsbuffer);
> -
> - [call_as(GetColumnInfo)]
> - HRESULT RemoteGetColumnInfo([in, out] DBORDINAL *columns, [out, size_is((ULONG)*columns)] DBCOLUMNINFO  **col_info,
> -
>    [out, size_is((ULONG)*columns)] DBBYTEOFFSET **name_offsets,
> -
>    [out, size_is((ULONG)*columns)] DBBYTEOFFSET **columnid_offsets,
> -
>    [in, out] DBLENGTH *string_len, [in, out, unique, size_is((ULONG)*string_len)] OLECHAR **stringsbuffer,
> -
>    [out] IErrorInfo **error);
> -

ULONG in SDK comes from DBORDINAL being ULONGLONG vs ULONG depending on
bitness, I agree we probably don't have to duplicate that, and can
always use cast. In any case size_is attribute is used in a wrong way,
and at least one method prototype does not match SDK.