[Cbc] Fix for uninitialized member variable _noNames in CoinModel

[victor.zverovich at gmail.com]

Hi Ted,

Thank you for a reply.

AFAICS this is not fixed in trunk yet - noNames_ is not initialized
anywhere in the CoinModel ctor at
https://projects.coin-or.org/Cbc/browser/trunk/Cbc/src/Cbc_ampl.cpp#L733

The version number I updated corresponds to the driver itself, it shouldn't
match the version of CBC. It is in the format used by other AMPL drivers
and can be obtained with the following command:

$ cbc -v -AMPL
CBC 2.8.1 (Linux x86_64), driver(20130502), ASL(20130419)

Best regards,
Victor

On Thu, May 2, 2013 at 8:55 PM, Ted Ralphs <ted at lehigh.edu> wrote:

> Hi Victor,
>
> What version are you using? This seems to be fixed already in trunk.
>
> The version number you updated is from an older versioning scheme. It
> would be nice to make it match the real CBC version number, but this is not
> actually a "number" and has hence been added to the solver name (see
> below). Do you happen to know if the version number you updated is
> necessary and if not, will replacing it with a zero suppress its use?
>
> static Option_Info Oinfo = {
>     const_cast<char*>("cbc"),
>     const_cast<char*>("CBC " CBC_VERSION),
>     const_cast<char*>("cbc_options"),
>     keywds,
>     nkeywds,
>     0,
>     0,
>     0,
>     decodePhrase,
>     0,
>     0,
>     0,
>     20070606
> };
>
> Ted
>
> On Thu, May 2, 2013 at 7:55 PM, victor.zverovich at gmail.com <
> victor.zverovich at gmail.com> wrote:
>
>> Hi,
>>
>> There is an assertion failure in debug version of cbc compiled with MSVC
>> when running the following AMPL code:
>>
>> var x >= 0;
>> minimize o: x * x;
>> option solver cbc;
>> solve;
>>
>> > ampl test.ampl
>>
>> Assertion failed: !noNames_, file CoinUtils\src\CoinModel.cpp, line 1251
>>
>> The attached patch fixes the problem by initializing the member variable
>> noNames_.
>>
>> Also the version number is updated.
>>
>> Best regards,
>> Victor
>>
>> _______________________________________________
>> Cbc mailing list
>> Cbc at list.coin-or.org
>> http://list.coin-or.org/mailman/listinfo/cbc
>>
>>
>
>
> --
> Dr. Ted Ralphs
> Associate Professor, Lehigh University
> (610) 628-1280
> ted 'at' lehigh 'dot' edu
> coral.ie.lehigh.edu/~ted
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://list.coin-or.org/pipermail/cbc/attachments/20130502/254c176f/attachment.html>