On 6/14/2013 14:12, Alistair Leslie-Hughes wrote:
> +typedef struct ErrorInfoImpl
> +{
> + IErrorInfo IErrorInfo_iface;
> + LONG ref;
> +
> + GUID m_Guid;
> + WCHAR *source;
> + WCHAR *description;
> + WCHAR *help_file;
> + DWORD m_dwHelpContext;
> +} ErrorInfoImpl;
Please use consistent naming, for example avoid prefixed names.
Also it would be cleaner to use BSTR here, but it will work either way.
> + TRACE("(%p)->(pBstrSource=%p)\n",This,pBstrSource);
No need for parameter names in traces.