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

Fix javadoc comments errors and warning reported by doclint report

        Details

        • Subcomponent:
        • Resolved In Build:
          b112

          Backports

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            cl Christine Lu added a comment -
            Sent a code review last week: https://sthinfra10.se.oracle.com/cru/CR-JDK8AWT-6

            But then found out SPB's fixes were pushed to awt forest, not tl forest, and we have lots of duplicate. To avoid duplicate and future merge conflict, I started all over in awt forest.

            Sent a new code review after switching to awt forest: https://sthinfra10.se.oracle.com/cru/CR-JDK8AWT-8. In this code review, 501 errors/warnings are fixed.
            Show
            cl Christine Lu added a comment - Sent a code review last week: https://sthinfra10.se.oracle.com/cru/CR-JDK8AWT-6 But then found out SPB's fixes were pushed to awt forest, not tl forest, and we have lots of duplicate. To avoid duplicate and future merge conflict, I started all over in awt forest. Sent a new code review after switching to awt forest: https://sthinfra10.se.oracle.com/cru/CR-JDK8AWT-8 . In this code review, 501 errors/warnings are fixed.
            Hide
            Permalink
            anthony Anthony Petrov (Inactive) added a comment -
            This should be reviewed on the awt-dev@ mailing list.
            Show
            anthony Anthony Petrov (Inactive) added a comment - This should be reviewed on the awt-dev@ mailing list.
            Hide
            Permalink
            cl Christine Lu added a comment -
            Re-posted the code review on http://cr.openjdk.java.net/~cl/8025409/webrev.00/, and email to awt-dev@ mailing list.
            Show
            cl Christine Lu added a comment - Re-posted the code review on http://cr.openjdk.java.net/~cl/8025409/webrev.00/, and email to awt-dev@ mailing list.
            Hide
            Permalink
            cl Christine Lu added a comment -
            Not sure when awt will sync to tl forest and tl to master. So I put the resolve in build b114.
            Thanks Anthony and Yuri for reviewing. I have pushed to awt forest.
            Show
            cl Christine Lu added a comment - Not sure when awt will sync to tl forest and tl to master. So I put the resolve in build b114. Thanks Anthony and Yuri for reviewing. I have pushed to awt forest.
            Hide
            Permalink
            hgupdate HG Updates added a comment -
            URL: http://hg.openjdk.java.net/jdk8/awt/jdk/rev/1da5d306e84b
            User: cl
            Date: 2013-10-02 18:29:10 +0000
            Show
            hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/jdk8/awt/jdk/rev/1da5d306e84b User: cl Date: 2013-10-02 18:29:10 +0000
            Hide
            Permalink
            anthony Anthony Petrov (Inactive) added a comment -
            Christine: you shouldn't mark bugs as Resolved manually. The HG Updates robot does this for you. Also, the "Resolved in Build" filed will be updated by an integrator once the fix makes it to the master repo. No need to update it manually.
            Show
            anthony Anthony Petrov (Inactive) added a comment - Christine: you shouldn't mark bugs as Resolved manually. The HG Updates robot does this for you. Also, the "Resolved in Build" filed will be updated by an integrator once the fix makes it to the master repo. No need to update it manually.
            Hide
            Permalink
            hgupdate HG Updates added a comment -
            URL: http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/1da5d306e84b
            User: lana
            Date: 2013-10-11 18:24:46 +0000
            Show
            hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/1da5d306e84b User: lana Date: 2013-10-11 18:24:46 +0000

              People

              • Assignee:
                cl Christine Lu
                Reporter:
                cl Christine Lu
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: