<div dir="ltr"><div>Dear Bruno,</div><div><br></div><div>thank you for looking into this. I have moved on from CLN and GiNaC on Windows, because I felt it was unmaintained and there was no interest working with me on upstreaming my patch. I hope this time things will work out differently.</div><div><br></div><div>Bests,</div><div>Robert<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Oct 27, 2019 at 6:56 PM Bruno Haible <<a href="mailto:bruno@clisp.org">bruno@clisp.org</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">Hi Robert,<br>
<br>
On 2018-02-24 you resent your mail and patches from 2012-08-15:<br>
<br>
> Here is the previous win64 patch set...<br>
> ---------- Forwarded message ----------<br>
> From: "Robert Szalai" <<a href="mailto:robicjedi@gmail.com" target="_blank">robicjedi@gmail.com</a>><br>
> Date: Aug 15, 2012 02:22<br>
> Subject: Re: [CLN-list] MinGW64 build<br>
> To: "CLN discussion list" <<a href="mailto:cln-list@ginac.de" target="_blank">cln-list@ginac.de</a>><br>
> Cc:<br>
> <br>
> Hi Alexei,<br>
> ><br>
> > I've splitted the path into 10 parts. There is a bigger one that is only<br>
> > s/long/intptr_t/  and s/unsigned long/uintptr_t/<br>
> > the rest is very small.<br>
> ><br>
> > Hopefully this can now be applied.<br>
> ><br>
> > > Ditto (the corresponding hunk is not quite optimal, though. {s,u}int32<br>
> > > can be typedef'ed to {,u}int32_t => no need for ugly #if's).<br>
> ><br>
> > I was thinking of this, however we need exactly the same underlying type as<br>
> > (u)intptr_t for the same sized integer to avoid compiler confusion.<br>
> > So in some cases (e.g. on i686 Linux) stdint.h might define int32_t as int<br>
> > and intptr_t as long<br>
> > and in this case the compiler will complain about multiple definition of<br>
> > the same function.<br>
> ><br>
> > Bests,<br>
> > Robert<br>
<br>
I have applied the 3 parts of your series that were good, and fixed the<br>
remaining problems. Indeed, the problems with the overload conflicts<br>
(in cl_combine, fprintdecimal, fprinthexadecimal) that are caused by the<br>
fact that intptr_t may be 'int', may be 'long', may be 'long long', were<br>
hardest to resolve. I chose not to introduce platform-dependent #ifs<br>
(too hairy to maintain), nor an autoconf test (because the include files<br>
are supposed to be usable by different compilers, and different compilers<br>
may define intptr_t differently), but instead use an inline function<br>
redirection.<br>
<br>
Bruno<br>
<br>
</blockquote></div>