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.