<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">On 3/11/2013 13:41, Jeff Latimer wrote:<br>
</div>
<blockquote cite="mid:513DA6D5.9070903@yless4u.com.au" type="cite">
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
<div class="moz-cite-prefix">On 11/03/13 17:43, Nikolay Sivov
wrote:<br>
</div>
<blockquote cite="mid:513D7D0F.7010500@gmail.com" type="cite">On
3/11/2013 10:27, Jeff Latimer wrote: <br>
<blockquote type="cite">--- <br>
include/nserror.h | 226 <br>
++++++++++++++++++++++++++++++++++++++++++++++++++++++ <br>
1 file changed, 226 insertions(+) <br>
create mode 100644 include/nserror.h <br>
+#ifndef _NSERROR_H <br>
+#define _NSERROR_H <br>
</blockquote>
This doesn't look right. <br>
</blockquote>
This is the same as the SDK, atleast to my eye. Is there
something in particular that appears wrong?<br>
</blockquote>
See how it's done in existing headers.<br>
<blockquote cite="mid:513DA6D5.9070903@yless4u.com.au" type="cite">
<blockquote cite="mid:513D7D0F.7010500@gmail.com" type="cite"> <br>
<blockquote type="cite">+#define NS_S_CALLPENDING
_HRESULT_TYPEDEF_(0x000D0000L) <br>
</blockquote>
This is usually done with a suffix. <br>
</blockquote>
I have done this as the SDK does it. I thought that was the way
we liked it.<br>
</blockquote>
Sorry, I meant to say 'without a suffix'. See how
_HRESULT_TYPEDEF_() is used in winerror.h for example.<br>
<blockquote cite="mid:513DA6D5.9070903@yless4u.com.au" type="cite">
<blockquote cite="mid:513D7D0F.7010500@gmail.com" type="cite"> <br>
<blockquote type="cite">+ <br>
+#endif <br>
</blockquote>
Please add a comment here.<br>
</blockquote>
Ok. </blockquote>
<br>
</body>
</html>