[Coin-ipopt] Simple code changes

Andreas Waechter andreasw at watson.ibm.com
Wed Oct 12 17:20:34 EDT 2005


Hi Ivan,

What you suggest makes sense; based on our offline discussion this
afternoon I realized that <cstdio> was NOT include via some other header
for the LinAlg files that you list, so I added the inclusion of <cstdio>
(not <iostream> though).  I also added the const qualifier.

I submitted your requested changes to the dev version.  (Keep in mind that
the dev version is subject to constant changes; we will derive new
official releases from time to time - since so far we didn't get any
serious bug reports for the current official release, I didn't see an
urgent reason yet to make a new official release... :)

Thanks for bringing this compatibility issue to our attention.

Best,

Andreas

On Wed, 12 Oct 2005, Oliveira, Ivan B wrote:

> Hi guys,
>
>
>
> We've used IPOPT with our own version of STL, which has uncovered a few
> places in the linear algebra code files where more explicit code would
> be helpful.  Please consider the following simple modifications to the
> code.  We've tested them out for both standard STL that ships with gcc
> and our own version, and it work for both.
>
>
>
> IpTripletToCSRConverter.hpp
>
>         -      bool operator< (const TripletEntry& Tentry)
>         +      bool operator< (const TripletEntry& Tentry) const
>
>
>
> IpCompoundMatrix.cpp
>
>         +#include <iostream>
>
>
>
> IpCompoundSymMatrix.cpp
>
>         +#include <iostream>
>
>
>
> IpCompoundVector.cpp
>
>         +#include <iostream>
>
>
>
> IpSumMatrix.cpp
>
>         +#include <iostream>
>
>
>
> IpSumSymMatrix.cpp
>
>         +#include <iostream>
>
>
>
> Thanks,
>
>
>
> --Ivan.
>
>
>
> ----------------------------
>
> Ivan B. Oliveira
>
> SC12-205
>
> (408)765-0584
>
>
>
>




More information about the Coin-ipopt mailing list