[Coin-discuss] Makefile.CYGWIN

[Matthew Saltzman]

On Tue, 5 Jul 2005, Matthew Galati wrote:

> Hi Matt,
>
> No patch or test done by me.
>
> Just suggesting to add those def's to the build (as in Linux), since some 
> COIN modules are heavy on assertions and it could be an unnecessary 
> performance penalty under -O.

It's a good suggestion, I'd just rather have somebody try the obvious 
patch before I commit it, to make sure it doesn't break anything.  I don't 
have a Cygwin installation handy ATM.  I wouldn't expect any problems, but 
would prefer to be prudent.

>
> Matt
>
> Matthew Galati - Optimization Developer
> SAS Institute - Analytical Solutions
> Phone 919-531-0332, R5327 Fax   919-677-4444
> http://coral.ie.lehigh.edu/~magh
> http://ordlnx2.na.sas.com/projects/OptWiki
> http://www.sas.com/technologies/analytics/optimization/
>
>
>
>
>> On Mon, 4 Jul 2005, Matthew Galati wrote:
>> 
>>> Is there any reason why Makefile.CYGWIN does not define -DNDEBUG and 
>>> -DCOIN_ASSERT when OptLevel=-O, as in Makefile.Linux?
>> 
>> 
>> I doubt it.  Probably somebody thought it would be a good idea for Linux 
>> and didn't think about--or couldn't test--Cygwin.  I can't tell from the 
>> log who made that particular change.
>> 
>> Do you have a tested patch?
>> 
>>> 
>>> Thanks,
>>> Matt
>>> 
>>> 
>> 
> _______________________________________________
> Coin-discuss mailing list
> Coin-discuss at list.coin-or.org
> http://list.coin-or.org/mailman/listinfo/coin-discuss
>

-- 
 		Matthew Saltzman

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