
<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Feb 5, 2015, at 1:46 PM, Austin English <<a href="mailto:austinenglish@gmail.com" class="">austinenglish@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">On Thu, Feb 5, 2015 at 2:40 PM, Charles Davis <span dir="ltr" class=""><<a href="mailto:cdavis5x@gmail.com" target="_blank" class="">cdavis5x@gmail.com</a>></span> wrote:<br class=""><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 class="">

> On Feb 5, 2015, at 1:20 PM, Austin English <<a href="mailto:austinenglish@gmail.com" class="">austinenglish@gmail.com</a>> wrote:<br class="">

> +typedef unsigned long ASN1uint32_t;<br class="">

> +typedef signed long ASN1int32_t;<br class="">

This won’t do what you (apparently) expect on 64-bit. At the very least, you should use ULONG/DWORD and LONG here.<br class="">

<br class="">

> +#ifndef WINAPI<br class="">

> +#define WINAPI __stdcall<br class="">

> +#endif<br class="">

This is most certainly wrong for Wine. I think you should include “windef.h” if you need WINAPI.<br class=""></blockquote><div class=""><br class=""></div><div class="">The same definition is made in include/bcrypt.h (I considered removing it, then saw that it was already in bcrypt.h).<br class=""></div></div></div></div></div></blockquote>Well I’ll be darned.</div><div><br class=""></div><div>I realized right after I replied to you that we define <font face="Bitstream Vera Sans Mono" class="">__stdcall</font> to a macro anyway, so I guess this is OK.<br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><div class=""> <br class=""></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

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 class=""></blockquote><div class=""><br class=""></div><div class="">No, mingw:<br class=""><a href="https://github.com/Alexpux/mingw-w64/blob/master/mingw-w64-headers/include/msasn1.h" class="">https://github.com/Alexpux/mingw-w64/blob/master/mingw-w64-headers/include/msasn1.h</a> <br class=""></div></div><a href="https://gitorious.org/mingw/mingw-w32api/source/4c8646e262e08f79e290f5211ab9e7386a85c753:include/ddk/hidpi.h" class="">https://gitorious.org/mingw/mingw-w32api/source/4c8646e262e08f79e290f5211ab9e7386a85c753:include/ddk/hidpi.h</a><br class=""></div></div></div></blockquote>I’m sorry for taking such an accusatory tone. It’s just that, the use of <font face="Bitstream Vera Sans Mono" class="">IN</font>/<font face="Bitstream Vera Sans Mono" class="">OUT</font> qualifiers (in <font face="Bitstream Vera Sans Mono" class=""><hidpi.h></font>) and Hungarian notation (here) made me suspicious. I guess if I have a beef with anyone, it’s MinGW ;).</div><div><br class=""></div><div>Chip</div><div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_extra"><br class=""></div><div class="gmail_extra">-- <br class=""><div class="gmail_signature">-Austin</div>

</div></div>

</div></blockquote></div><br class=""></body></html>