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.