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 #599 from BBasile/issue-575 

fix #575 - Dscanner should allow opCmp with no toHash
This commit is contained in:
Sebastian Wilzbach 2018-04-05 11:56:51 +02:00 committed by GitHub
parent a4d84a0995 6fe77da02d
commit 9df55a7de2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/dscanner/analysis/opequals_without_tohash.d
View File

@ -76,12 +76,6 @@ class OpEqualsWithoutToHashCheck : BaseAnalyzer
string message = "'" ~ name.text ~ "' has method 'toHash', but not 'opEquals'.";
addErrorMessage(name.line, name.column, KEY, message);
}
if (hasOpCmp && !hasOpEquals)
{
addErrorMessage(name.line, name.column, KEY,
"'" ~ name.text ~ "' has method 'opCmp', but not 'opEquals'.");
}
}
enum string KEY = "dscanner.suspicious.incomplete_operator_overloading";
@ -108,6 +102,15 @@ unittest
}
}
// AA would use default equal and default toHash
struct Bee
{
int opCmp(Bee) const
{
return true;
}
}
// Fail on class opEquals
class Rabbit // [warn]: 'Rabbit' has method 'opEquals', but not 'toHash'.
{