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