JDK-8249633: doclint reports missing javadoc for JavaFX property methods that have a property description

[jonathan-gibbons]

There are two parts to the fix:

1. Similar to that of JDK-8251200: predicate methods in Utils used to check if an element has certain tags in its doc comment may trigger the missing comment check even when there does not need to be a comment.

2. When synthesizing comments for the property-related methods, the code checks for the existence of explicit comments, in case they exist; that could also trigger the "missing comment" check.

The fix shares some changes with JDK-8251200; whichever does not go first will need a merge commit.

---

Progress

• Change must not contain extraneous whitespace
• Commit message must refer to an issue
• Change must be properly reviewed

Issue

• JDK-8249633: doclint reports missing javadoc for JavaFX property methods that have a property description

Reviewers

• Hannes Wallnöfer (@hns - Reviewer) ⚠️ Review applies to 39e1ec7

Download

$ git fetch https://git.openjdk.java.net/jdk16 pull/71/head:pull/71
$ git checkout pull/71

[bridgekeeper]

👋 Welcome back jjg! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

[openjdk]

@jonathan-gibbons The following label will be automatically applied to this pull request:

• javadoc

When this pull request is ready to be reviewed, an "RFR" email will be sent to the corresponding mailing list. If you would like to change these labels, use the /label pull request command.

[mlbridge]

Webrevs

• 02: Full - Incremental (67519b8)
• 01: Full (2f995c1)
• 00: Full (39e1ec7)

[hns]

Looks good, same two typos as in previous PR :-)

[hns]

typo: copmment

[hns]

typo: eleme nt

[openjdk]

@jonathan-gibbons This change now passes all automated pre-integration checks.

ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details.

After integration, the commit message for the final commit will be:

8249633: doclint reports missing javadoc for JavaFX property methods that have a property description

Reviewed-by: hannesw

You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed.

At the time when this comment was updated there had been no new commits pushed to the master branch. If another commit should be pushed before you perform the /integrate command, your PR will be automatically rebased. If you prefer to avoid any potential automatic rebasing, please check the documentation for the /integrate command for further details.

➡️ To integrate this PR with the above commit message to the master branch, type /integrate in a new comment.

[jonathan-gibbons]

/integrate

[openjdk]

@jonathan-gibbons Pushed as commit 4f914e2.

💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored.