Re: [PATCH 0/2] MR2709: include: Add ISpTTSEngineSite and ISpTTSEngine interfaces.

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about include/sapiddk.idl:
> - HRESULT GetActions();
> - HRESULT Write([in] const void *pBuff,
> -
>              [in] ULONG cb,
> -
>              [out] ULONG *pcbWritten);
> - HRESULT GetRate([out] long *pRateAdjust);
> - HRESULT GetVolume([out] USHORT *pusVolume);
> - HRESULT GetSkipInfo([out] SPVSKIPTYPE *peType,
> -
>                    [out] long *plNumItems);
> - HRESULT CompleteSkip([in] long lNumSkipped);
>
> +};
> -
>
> +typedef struct SPVTEXTFRAG
> +{
> - struct SPVTEXTFRAG* pNext;
> - SPVSTATE  State;
> - LPCWSTR  pTextStart;

The alignment of the 'p' looks a bit odd - I'd suggest an extra space.