Incorrect code in jdk.javadoc.internal.doclets.toolkit.util. VisibleMemberMap. ClassMembers

XMLWordPrintable

    • Type: Bug
    • Resolution: Not an Issue
    • Priority: P4
    • tbd
    • Affects Version/s: 9
    • Component/s: tools

      Incorrect code in jdk.javadoc.internal.doclets.toolkit.util.VisibleMemberMap.ClassMembers.

      "tagName" variable is compared with strings erroneously prefixed by "@":

      private void checkOnPropertiesTags(List<Element> members) {
      ............................................................................................
                              if (tagName.equals("@propertySetter")
                                      || tagName.equals("@propertyGetter")
                                      || tagName.equals("@propertyDescription")) {
                                  if (!isPropertyGetterOrSetter(members,
      ............................................................................................

      "propertyMethodName" is erroneously suffixed by "Property" making the following comparison senseless:

      private boolean isPropertyGetterOrSetter(List<Element> members,
                                                       ExecutableElement method) {
                  String propertyName = utils.propertyNameFromMethodName(utils.getSimpleName(method));
                  if (!propertyName.isEmpty()) {
                      String propertyMethodName = propertyName + "Property";
                      for (Element member: members) {
                          if (utils.getSimpleName(member).equals(propertyMethodName)) {
                              return true;
                          }
      ............................................................................................

            Assignee:
            Unassigned
            Reporter:
            Oleg Barbashov (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: