<div dir="ltr">On Thu, Feb 5, 2015 at 3:55 PM, Charles Davis <span dir="ltr"><<a href="mailto:cdavis5x@gmail.com" target="_blank">cdavis5x@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><span class=""><blockquote type="cite"><div>On Feb 5, 2015, at 1:46 PM, Austin English <<a href="mailto:austinenglish@gmail.com" target="_blank">austinenglish@gmail.com</a>> wrote:</div><br><div><div dir="ltr">On Thu, Feb 5, 2015 at 2:40 PM, Charles Davis <span dir="ltr"><<a href="mailto:cdavis5x@gmail.com" target="_blank">cdavis5x@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
> On Feb 5, 2015, at 1:20 PM, Austin English <<a href="mailto:austinenglish@gmail.com" target="_blank">austinenglish@gmail.com</a>> wrote:<br>
> +typedef unsigned long ASN1uint32_t;<br>
> +typedef signed long ASN1int32_t;<br>
This won’t do what you (apparently) expect on 64-bit. At the very least, you should use ULONG/DWORD and LONG here.<br>
<br>
> +#ifndef WINAPI<br>
> +#define WINAPI __stdcall<br>
> +#endif<br>
This is most certainly wrong for Wine. I think you should include “windef.h” if you need WINAPI.<br></blockquote><div><br></div><div>The same definition is made in include/bcrypt.h (I considered removing it, then saw that it was already in bcrypt.h).<br></div></div></div></div></div></blockquote></span>Well I’ll be darned.</div><div><br></div><div>I realized right after I replied to you that we define <font face="Bitstream Vera Sans Mono">__stdcall</font> to a macro anyway, so I guess this is OK.<span class=""><br></span> <br></div></div></blockquote><div><br></div><div>I don't mind changing it, as you pointed out, it's probably not needed.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><div style="word-wrap:break-word"><div>I’m beginning to wonder if you copied this header (as well as <hidpi.h>) directly from Microsoft’s SDKs. You’ve been around long enough to know that that’s a big no-no around here.<br><span class=""><blockquote type="cite"><div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>No, mingw:<br><a href="https://github.com/Alexpux/mingw-w64/blob/master/mingw-w64-headers/include/msasn1.h" target="_blank">https://github.com/Alexpux/mingw-w64/blob/master/mingw-w64-headers/include/msasn1.h</a> <br></div></div><a href="https://gitorious.org/mingw/mingw-w32api/source/4c8646e262e08f79e290f5211ab9e7386a85c753:include/ddk/hidpi.h" target="_blank">https://gitorious.org/mingw/mingw-w32api/source/4c8646e262e08f79e290f5211ab9e7386a85c753:include/ddk/hidpi.h</a><br></div></div></div></blockquote></span>I’m sorry for taking such an accusatory tone. It’s just that, the use of <font face="Bitstream Vera Sans Mono">IN</font>/<font face="Bitstream Vera Sans Mono">OUT</font> qualifiers (in <font face="Bitstream Vera Sans Mono"><hidpi.h></font>) and Hungarian notation (here) made me suspicious. I guess if I have a beef with anyone, it’s MinGW ;).</div></div></blockquote></div><br clear="all"></div><div class="gmail_extra">No worries, I wanted to minimize the overall changes between the two versions, aside from fixed indentation and removing mingw specific comments.<br><br></div><div class="gmail_extra">-- <br><div class="gmail_signature">-Austin</div>
</div></div>