<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Apr 2, 2016 at 1:59 PM, Steve Mitchell <span dir="ltr"><<a href="mailto:mitchell@intertrust.com" target="_blank">mitchell@intertrust.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div link="blue" vlink="purple" lang="EN-US">
<div>
<p class="MsoNormal">I have noticed a few issues with the “rational” custom C dtype example.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p><u></u><span>1.<span style="font:7.0pt "Times New Roman"">      
</span></span><u></u> It doesn’t build on Windows.  I managed to tweak it to build.  Mainly, the MSVC9 compiler is C89.<u></u><u></u></p>
<p><u></u><span>2.<span style="font:7.0pt "Times New Roman"">      
</span></span><u></u>A few tests don’t pass on Windows, due to integer sizes.<u></u><u></u></p>
<p><u></u><span>3.<span style="font:7.0pt "Times New Roman"">      
</span></span><u></u>The copyswap and copyswapn routines don’t do in-place swapping if src == NULL, as specified in the docs.<br>
<a href="http://docs.scipy.org/doc/numpy-1.10.0/reference/c-api.types-and-structures.html" target="_blank">http://docs.scipy.org/doc/numpy-1.10.0/reference/c-api.types-and-structures.html</a><span class="HOEnZb"><font color="#888888"><u></u><u></u></font></span></p><span class="HOEnZb"><font color="#888888">
<p class="MsoNormal"><u></u> </p></font></span></div></div></blockquote><div><br></div><div>Needless to say the example hasn't been tested on windows ;) A PR fixing the issues that you discovered would be welcome.<br><br></div><div>Chuck <br></div></div></div></div>