[D-runtime] change 78, make files

Sean Kelly sean at invisibleduck.org
Wed Jan 21 11:20:21 PST 2009


The ones you emailed me a week or two ago.  I finally checked them in 
yesterday.

As for unit tests... that's kind of weird.  I think the correct thing is 
to not have either -release /or/ -debug set, since you don't want the 
debug blocks activated, nor do you want to lose asserts (or contracts, I 
suppose).  I struggled with this for druntime, and finally decided that 
the "debug" build would not set any flags, including debug, and the 
"release" build would set the release flag.  The idea being that the 
build should be used for someone to debug their own code, so debug 
blocks within the lib itself weren't appropriate, but debug symbols, 
contracts, etc, were.  However, I think I subsequently forgot about this 
decision and made the unittest build link the release lib.

Andrei Alexandrescu wrote:
> Which makefiles?
> 
> Andrei
> 
> Brad Roberts wrote:
>> A great step forward.. better than what I'd done (I was trying to make
>> minimal changes rather than overhauling) and matches what phobos has,
>> for the most part.
>>
>> One concern.. the unit tests are based on release mode, which is rather
>> problematic.  Release mode disables asserts -- ie nothing is really
>> tested. :)
>>
>> Sean, I'll re-look at my changes, but between this one and the change
>> you're already started on (getting the sync primitives to build) are all
>> I had, I think.
>>
>> Later,
>> Brad
>>
> _______________________________________________
> D-runtime mailing list
> D-runtime at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/d-runtime



More information about the D-runtime mailing list