<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>Hi,</p>
<p>I reviewed Robert's patches and added some of my own, that were
necessary for GiNaC to work with CLN on Win64. They can be viewed
on Github:</p>
<p>=== CLN ===<br>
</p>
<p><a class="moz-txt-link-freetext" href="https://github.com/jrheinlaender/cln/commits/win64">https://github.com/jrheinlaender/cln/commits/win64</a></p>
<p>There are two kinds of patches:</p>
<p>- Patches required for Win64 platform (e.g. UL is 32 bit type on
that platform). Mostly due to Robert (Number 1-10)<br>
</p>
<p>- Cosmetic patches to avoid unnecessary compiler warnings (e.g.
confusion of class and struct)</p>
<p>Most patches I surrounded with #ifdef _M_AMD64 so that other
platforms will be unaffected<br>
</p>
<p>=== GiNaC ===<br>
</p>
<p><a class="moz-txt-link-freetext" href="https://github.com/jrheinlaender/ginac/commits/win64">https://github.com/jrheinlaender/ginac/commits/win64</a></p>
<p>- Here I removed some old patches that are not required any more
because of improvements in MSVC 2015</p>
<p>- And added a few constructors from size_t <br>
</p>
<p>I hope these patches are OK and can be included in master.<br>
</p>
<br>
<div class="moz-cite-prefix">Am 24.02.2018 um 12:49 schrieb Robert
Szalai:<br>
</div>
<blockquote type="cite"
cite="mid:CADe39dqhsbtq1DuuR2EfdrbwrQBS-g31rGDQM9ZgMRy50osiAg@mail.gmail.com">
<p dir="ltr">Here is the previous win64 patch set...</p>
<div class="gmail_quote">---------- Forwarded message ----------<br>
From: "Robert Szalai" <<a href="mailto:robicjedi@gmail.com"
moz-do-not-send="true">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"
moz-do-not-send="true">cln-list@ginac.de</a>><br>
Cc: <br>
<br type="attribution">
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">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 and intptr_t as long<br>
and in this case the compiler will complain about multiple
definition of the same function.<br>
<br>
Bests,<br>
Robert<br>
</blockquote>
</div>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
CLN-list mailing list
<a class="moz-txt-link-abbreviated" href="mailto:CLN-list@ginac.de">CLN-list@ginac.de</a>
<a class="moz-txt-link-freetext" href="https://www.cebix.net/mailman/listinfo/cln-list">https://www.cebix.net/mailman/listinfo/cln-list</a></pre>
</blockquote>
<br>
</body>
</html>