mirrors/phobos
1
0
Fork 0
mirror of https://github.com/dlang/phobos.git synced 2025-05-04 00:54:05 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

DScanner: check for useless asserts

Browse source
This commit is contained in:
Sebastian Wilzbach 2017-02-20 15:10:09 +01:00
parent 395ae88ac7
commit 1552ed5e18
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
.dscanner.ini
View file

@ -63,7 +63,7 @@ auto_ref_assignment_check="disabled" ; FIXME
; Checks for incorrect infinite range definitions ; Checks for incorrect infinite range definitions
incorrect_infinite_range_check="enabled" incorrect_infinite_range_check="enabled"
; Checks for asserts that are always true ; Checks for asserts that are always true
useless_assert_check="skip-unittest" useless_assert_check="enabled"
; Check for uses of the old-style alias syntax ; Check for uses of the old-style alias syntax
alias_syntax_check="enabled" alias_syntax_check="enabled"
; Checks for else if that should be else static if ; Checks for else if that should be else static if

4
std/algorithm/comparison.d
View file

@ -417,7 +417,7 @@ auto castSwitch(choices...)(Object switchObject)
// Void handlers are also allowed if all the handlers are void: // Void handlers are also allowed if all the handlers are void:
new A().castSwitch!( new A().castSwitch!(
(A a) { assert(true); }, (A a) { },
(B b) { assert(false); }, (B b) { assert(false); },
)(); )();
} }
@ -485,7 +485,7 @@ auto castSwitch(choices...)(Object switchObject)
// All-void handlers work for the null case: // All-void handlers work for the null case:
null.castSwitch!( null.castSwitch!(
(Object o) { assert(false); }, (Object o) { assert(false); },
() { assert(true); }, () { },
)(); )();
// Throwing void handlers work for the null case: // Throwing void handlers work for the null case: