[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