<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Sep 27, 2019 at 10:48 AM Rafał Mużyło <<a href="mailto:galtgendo@o2.pl">galtgendo@o2.pl</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Fri, Sep 27, 2019 at 09:54:44AM +0300, Nikolay Sivov wrote:<br>
> Hi, Rafał.<br>
> <br>
> Thanks for the patch. I think there is at least two issues with it - on<br>
> Windows only 32-bit modules use decorated names, 64-bit ones do not. We<br>
> need to do the same. Another issue is that you change to stdcall for all<br>
> versions, and according to recent SDK at least version 1.7 is using cdecl.<br>
> I suspect you should switch to stdcall only for 0,1,2 versions (if testing<br>
> confirms that).<br>
<br>
OK.<br>
First is might be a significant problem - I'm not sure if spec file<br>
syntax allows for diferent values for x86 and x64. Where is it<br>
documented anyway ?<br></blockquote><div><br></div><div>It allows that, msvcp* is a good example where names are arch-dependent. I don't know if it's documented.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
As for the second, are you sure ? I haven't located those SDKs, but I<br>
did google 1.1 and 1.7 versions of that header and *both* were stdcals.<br></blockquote><div><br></div><div>Headers I have from 10.0.17763.0 have those as STDAPIVCALLTYPE, which is cdecl.</div><div> </div></div></div>