DIP 1006 - Preliminary Review Round 1

[Walter Bright]

On 3/3/2018 5:29 PM, Nicholas Wilson wrote:
>> The use of comma-separated arguments is something I've argued against for 
>> other switches. The use of `-release=in -release=out` should be fine and is 
>> less confusing/buggy to implement.
> Why?  Implementation is trivial (unit testable no less!) , see 
> https://github.com/dlang/dmd/pull/7863 and extensible to other args.

It's not trivial. First, it's 100 lines of code (with no comments) just for 
that, and there are templates and behaviors with memory allocation. Moving along 
that path, we're gradually reinventing std.getopt which is 1814 lines (with 
comments).

Generally, people will be driving dmd with a makefile, dmd.conf, or other 
response file. It's complicated enough already, and gets constantly worse. Once 
in, we're stuck with it forever. I just don't feel it is worthwhile spending 
time on this.


> I think aggregation is a good idea. So is phrasing in the positive sense, except 
> for it looks like xxx and yyy are _off_ by default, where surely we want all 
> checks _on_ by default.

The default is that they're all on. So to just have one on, first turn them all 
off then turn on the desired ones.