Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8252310

Fix "no comment" warnings in java.naming

    XMLWordPrintable

    Details

    • Type: CSR
    • Status: Closed
    • Priority: P4
    • Resolution: Approved
    • Fix Version/s: 16
    • Component/s: core-libs
    • Labels:
      None
    • Subcomponent:
    • Compatibility Risk:
      minimal
    • Compatibility Risk Description:
      Hide
      There is no change to the behavior, the javadoc comments will appear in the javadoc.
      Show
      There is no change to the behavior, the javadoc comments will appear in the javadoc.
    • Interface Kind:
      Java API
    • Scope:
      SE

      Description

      Summary

      Address the "no comment" warnings in java.naming generated by javadoc -Xdoclint

      Problem

      java.naming contains the following classes with fields and/or methods which are missing a javadoc comment:

      • src/java.naming/share/classes/javax/naming/CompositeName.java
      • src/java.naming/share/classes/javax/naming/CompoundName.java
      • src/java.naming/share/classes/javax/naming/directory/BasicAttribute.java
      • src/java.naming/share/classes/javax/naming/directory/BasicAttributes.java
      • src/java.naming/share/classes/javax/naming/ldap/LdapName.java
      • src/java.naming/share/classes/javax/naming/ldap/Rdn.java
      • src/java.naming/share/classes/javax/naming/spi/NamingManager.java

      Solution

      Add the missing comments to the appropriate fields and methods in java.naming

      Specification

      Please see the attached mercurial diff patch (JDK-8251182.patch) and webrev (JDK-8251182.zip) for the added javadoc changes

        Attachments

        1. JDK-8251182.patch
          15 kB
        2. JDK-8251182.patch
          12 kB
        3. JDK-8251182.zip
          451 kB
        4. JDK-8251182.zip
          443 kB

          Issue Links

            Activity

              People

              Assignee:
              aefimov Aleksej Efimov
              Reporter:
              aefimov Aleksej Efimov
              Reviewed By:
              Lance Andersen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: