All asserts need to have messages attached! Explicit as possible!

Seb via Digitalmars-d digitalmars-d at puremagic.com
Sat Jul 8 08:36:00 PDT 2017


On Saturday, 8 July 2017 at 01:31:54 UTC, Seb wrote:
> On Saturday, 8 July 2017 at 01:01:38 UTC, Vladimir Panteleev 
> wrote:
>> On Saturday, 8 July 2017 at 00:55:46 UTC, FoxyBrown wrote:
>>> It would be easy to find all the bad asserts?
>>
>> Does Dscanner have a rule to detect assert and enforce calls 
>> without a message? We should have it enabled for Phobos.
>
> No, but now there's: 
> https://github.com/dlang-community/D-Scanner/pull/495
> I can always add this to the phobos branch at DScanner if you 
> want to test this immediately.

We can improve the status quo:

https://github.com/dlang/phobos/pull/5578

Once this is integrated, everyone can help out and improve Phobos 
by just removing a single module from the blacklist - it's a PR 
away.


More information about the Digitalmars-d mailing list