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.


Reply via email to