Add a hashCode() method to a class that already has equals()
Generally, overriding one means you should be overriding the other too. Also, configure Eclipse to treat any similar cases as errors, rather than merely warnings.
This commit is contained in:
parent
bb032c9869
commit
29f53d11fe
|
@ -39,7 +39,7 @@ org.eclipse.jdt.core.compiler.problem.methodWithConstructorName=error
|
|||
org.eclipse.jdt.core.compiler.problem.missingDefaultCase=error
|
||||
org.eclipse.jdt.core.compiler.problem.missingDeprecatedAnnotation=error
|
||||
org.eclipse.jdt.core.compiler.problem.missingEnumCaseDespiteDefault=disabled
|
||||
org.eclipse.jdt.core.compiler.problem.missingHashCodeMethod=warning
|
||||
org.eclipse.jdt.core.compiler.problem.missingHashCodeMethod=error
|
||||
org.eclipse.jdt.core.compiler.problem.missingJavadocComments=ignore
|
||||
org.eclipse.jdt.core.compiler.problem.missingJavadocCommentsVisibility=public
|
||||
org.eclipse.jdt.core.compiler.problem.missingJavadocTagDescription=no_tag
|
||||
|
|
|
@ -91,6 +91,11 @@ public class FakeExceptionTypeBinding implements ITypeBinding {
|
|||
return false;
|
||||
}
|
||||
|
||||
@Override
|
||||
public int hashCode() {
|
||||
return exceptionBinaryName.hashCode();
|
||||
}
|
||||
|
||||
// --- rest not needed
|
||||
|
||||
@Override
|
||||
|
|
|
@ -60,7 +60,7 @@ org.eclipse.jdt.core.compiler.problem.methodWithConstructorName=error
|
|||
org.eclipse.jdt.core.compiler.problem.missingDefaultCase=ignore
|
||||
org.eclipse.jdt.core.compiler.problem.missingDeprecatedAnnotation=error
|
||||
org.eclipse.jdt.core.compiler.problem.missingEnumCaseDespiteDefault=disabled
|
||||
org.eclipse.jdt.core.compiler.problem.missingHashCodeMethod=ignore
|
||||
org.eclipse.jdt.core.compiler.problem.missingHashCodeMethod=error
|
||||
org.eclipse.jdt.core.compiler.problem.missingJavadocComments=ignore
|
||||
org.eclipse.jdt.core.compiler.problem.missingJavadocCommentsOverriding=disabled
|
||||
org.eclipse.jdt.core.compiler.problem.missingJavadocCommentsVisibility=public
|
||||
|
|
Loading…
Reference in New Issue