diff --git a/feed.xml b/feed.xml index bdd9a6b3c2..5f73e66302 100644 --- a/feed.xml +++ b/feed.xml @@ -5,8 +5,8 @@ Intended as a documentation theme based on Jekyll for technical writers documenting software and other technical products, this theme has all the elements you would need to handle multiple products with both multi-level sidebar navigation, tags, and other documentation features. https://pmd.github.io/pmd/ - Tue, 25 Aug 2020 12:44:10 +0000 - Tue, 25 Aug 2020 12:44:10 +0000 + Tue, 25 Aug 2020 13:03:31 +0000 + Tue, 25 Aug 2020 13:03:31 +0000 Jekyll v3.9.0 diff --git a/pmd_release_notes.html b/pmd_release_notes.html index a65288dabe..f3430982b1 100644 --- a/pmd_release_notes.html +++ b/pmd_release_notes.html @@ -1516,6 +1516,7 @@ which was introduced in PMD 6.26.0.
  • java-errorprone diff --git a/pmd_rules_java.html b/pmd_rules_java.html index 5f64ed6a3f..71e658c641 100644 --- a/pmd_rules_java.html +++ b/pmd_rules_java.html @@ -1690,7 +1690,7 @@ $('#toc').toc({ minimumHeaders: 0, listType: 'ul', showSpeed: 0, headers: 'h2,h3
  • SingletonClassReturningNewInstance: Some classes contain overloaded getInstance. The problem with overloaded getInstance methods is t…
  • StaticEJBFieldShouldBeFinal: According to the J2EE specification, an EJB should not have any static fields with write access. …
  • StringBufferInstantiationWithChar: Individual character values provided as initialization arguments will be converted into integers….
  • -
  • SuspiciousEqualsMethodName: The method name and parameter number are suspiciously close to equals(Object), which can denote a…
  • +
  • SuspiciousEqualsMethodName: The method name and parameter number are suspiciously close to ‘Object.equals’, which can denote …
  • SuspiciousHashcodeMethodName: The method name and return type are suspiciously close to hashCode(), which may denote an intenti…
  • SuspiciousOctalEscape: A suspicious octal escape sequence was found inside a String literal. The Java language specifica…
  • TestClassWithoutTestCases: Test classes end with the suffix Test. Having a non-test class with that name is not a good pract…
  • diff --git a/pmd_rules_java_errorprone.html b/pmd_rules_java_errorprone.html index 68c8ecc96e..ee36540644 100644 --- a/pmd_rules_java_errorprone.html +++ b/pmd_rules_java_errorprone.html @@ -4671,8 +4671,12 @@ new StringBuilder("A") // 1 + 16 = 17

    Priority: Medium High (2)

    -

    The method name and parameter number are suspiciously close to equals(Object), which can denote an -intention to override the equals(Object) method.

    +

    The method name and parameter number are suspiciously close to Object.equals, which can denote an +intention to override it. However, the method does not override Object.equals, but overloads it instead. +Overloading Object.equals method is confusing for other programmers, error-prone and hard to maintain, +especially when using inheritance, because @Override annotations used in subclasses can provide a false +sense of security. For more information on Object.equals method, see Effective Java, 3rd Edition, +Item 10: Obey the general contract when overriding equals.

    This rule is defined by the following XPath expression:

    //MethodDeclaration[@Name = 'equals']