forks
/
D-Scanner
mirror of https://github.com/dlang-community/D-Scanner.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #373 from BBasile/uselessassert-skiput-doc 

fixup for #368, readme and source comment still stated the prev.behavior
This commit is contained in:
Brian Schott 2016-09-06 15:08:29 -07:00 committed by GitHub
parent 7fc17a0d58 f9bb76782c
commit 32f45847fd
2 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
README.md
View File

@ -115,7 +115,7 @@ Note that the "--skipTests" option is the equivalent of changing each
* Unused labels. * Unused labels.
* Lines longer than 120 characters. * Lines longer than 120 characters.
* Incorrect infinite range definitions. * Incorrect infinite range definitions.
* Some assertions that check conditions that will always be true. This check can't be skipped in the tests. * Some assertions that check conditions that will always be true.
#### Wishlist #### Wishlist

1
src/analysis/useless_assert.d
View File

@ -22,7 +22,6 @@ class UselessAssertCheck : BaseAnalyzer
/// ///
this(string fileName, bool skipTests = false) this(string fileName, bool skipTests = false)
{ {
// assertions likely to be in unittest so never skip
super(fileName, null, skipTests); super(fileName, null, skipTests);
} }