[Coin-discuss] minor bugs, CoinMpsIO, OsiSolverInterfaceTest

[Matthew Saltzman]

These are all fixed, as is inconsistent handling of BOUNDS (vs. BOUND) in
read and write.

On Thu, 26 Sep 2002, Lou Hafer wrote:

> Folks,
>
>
> 	A few simple fixes for CoinMpsIO.cpp and OsiSolverInterfaceTest.cpp.
>
> 							Lou
>
> -------------------------------------------------------------
>
>   CoinMpsIO: line numbers may be slightly off as I added some bits of
>   commentary while working. Shouldn't amount to more than five lines
>   plus/minus. In all cases, top line should replace lower line.
>
>   * line 175
>
>     <   MpsCardReader ( FILE * fp, gzFile gzfp, CoinMpsIO * reader );
>     ---
>     >   MpsCardReader ( FILE * fp, gzFile fp, CoinMpsIO * reader );
>
>     Two identically named parameters --- the compiler was unhappy.
>
>   * line 2367
>
>     <       writeString(fp, gzfp, "RANGES\n");
>     ---
>     >       writeString(fp, gzfp, "RANGE\n");
>
>     If I read the MPS standard correctly, RANGES is required as the section
>     name.
>
>   * line 2832
>
>     <   if ( intType[columnNumber]==1 ) return true;
>     ---
>     >   if ( intType[columnNumber]==0 ) return true;
>
>     Looks like an overly enthusiastic application of cut-and-paste code
>     construction  :-). Integer is indicated by `1', yes?
>
>
>   OsiSolverInterfaceTest.cpp
>
>   * line 19
>
>     < #  include "OsiDylpSolverInterface.hpp"
>     ---
>     > #  include "OsiDyLpSolverInterface.hpp"
>
>     The problem is the capital `L' in DyLp.
>
> _______________________________________________
> Coin-discuss mailing list
> Coin-discuss at www-124.ibm.com
> http://www-124.ibm.com/developerworks/oss/mailman/listinfo/coin-discuss
>

-- 
		Matthew Saltzman

Clemson University Math Sciences
mjs at clemson.edu
http://www.math.clemson.edu/~mjs