<div dir="ltr">Just to second Victor's comments, Python extensions must still be built with VC2008 and since the C interface is particularly useful for building such extensions, it would be great not to break compatibility with VC 2008. <div><br></div><div>I'm in the process of building CoinAll trunk with VC2008, but have run into some other difficulties I'm trying to find fixes for. I also have VC2013 and tried building with it as well, but there were problems there, too.<div><br></div><div>Cheers,</div><div><br></div><div>Ted</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 2, 2014 at 11:56 AM, Miles Lubin <span dir="ltr"><<a href="mailto:miles.lubin@gmail.com" target="_blank">miles.lubin@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Okay, I've committed the VC2010 workaround to trunk.<div><br></div><div>Thanks,</div><div>Miles</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 2, 2014 at 11:47 AM, <a href="mailto:victor.zverovich@gmail.com" target="_blank">victor.zverovich@gmail.com</a> <span dir="ltr"><<a href="mailto:victor.zverovich@gmail.com" target="_blank">victor.zverovich@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Miles,<div><br></div><div>Yes, I know that it is C99 which is not fully implemented by VC2010. However, CInterfaceTest is the only part of Cbc that requires it, so I think it would be beneficial not to unnecessarily break compatibility. I haven't try VC2013.</div><div><br></div><div>Cheers,</div><div>Victor</div></div><div><div class="h5"><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 2, 2014 at 8:41 AM, Miles Lubin <span dir="ltr"><<a href="mailto:miles.lubin@gmail.com" target="_blank">miles.lubin@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Victor,<div><br></div><div>Thanks for reporting this. I've included stdlib.h in trunk. Regarding INFINITY, this is a C99 feature that's supported in VC2013 (<a href="http://blogs.msdn.com/b/vcblog/archive/2013/07/19/c99-library-support-in-visual-studio-2013.aspx" target="_blank">http://blogs.msdn.com/b/vcblog/archive/2013/07/19/c99-library-support-in-visual-studio-2013.aspx</a>). Have you tried compiling with VC2013? I'm open to including the workaround if needed.</div><div><br></div><div>Best,</div><div>Miles</div></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 2, 2014 at 11:31 AM, <a href="mailto:victor.zverovich@gmail.com" target="_blank">victor.zverovich@gmail.com</a> <span dir="ltr"><<a href="mailto:victor.zverovich@gmail.com" target="_blank">victor.zverovich@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Cbc/test/CInterfaceTest.c fails to compile on VC2010 (and possibly other compilers) because the latter doesn't have the INFINITY macro:<br></div><div><br></div><div> Cbc\test\CInterfaceTest.c(43): error C2065: 'INFINITY' : undeclared identifier<br></div><div><br></div><div>Also stdlib.h should be included for malloc:</div><div><br></div><div> Cbc\test\CInterfaceTest.c(47): warning C4013: 'malloc' undefined; assuming extern returning int<br></div><div><br></div><div>Possible fix is available here: <a href="https://github.com/ampl/coin/commit/a260a3b01d3cbb24280636b514d1f7ca65b273f4" target="_blank">https://github.com/ampl/coin/commit/a260a3b01d3cbb24280636b514d1f7ca65b273f4</a></div><div><br></div><div>Best regards,</div><div>Victor</div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></div></div></blockquote></div><br></div>
<br>_______________________________________________<br>
Cbc mailing list<br>
<a href="mailto:Cbc@list.coin-or.org">Cbc@list.coin-or.org</a><br>
<a href="http://list.coin-or.org/mailman/listinfo/cbc" target="_blank">http://list.coin-or.org/mailman/listinfo/cbc</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>Dr. Ted Ralphs<br>Associate Professor, Lehigh University<br>(610) 628-1280<br>ted 'at' lehigh 'dot' edu<br><a href="http://coral.ie.lehigh.edu/~ted" target="_blank">coral.ie.lehigh.edu/~ted</a><br>
</div>