On Thu, 23 May 2024 15:22:38 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
> [JDK-8298405](https://bugs.openjdk.org/browse/JDK-8298405) added markdown > support to javadoc, using `///` to indicate markdown documentation comments. > As a result, building JavaFX docs with JDK 23 produces new warnings, which > causes the build to fail (since we treat warnings as errors). > > This PR was generated by running a program to automate the modification of > each line that contains three or more slash chars in a row, `///` (except > those in String literals). The replacement algorithm is as follows: > > 1. Replace each occurrence of exactly 3 slash chars `///` with 2 slash chars > `//` > 2. Replace each occurrence of 4 or more slash chars with an equal number of > `-` chars, leaving the first two `//` chars of the first run of `////` as is. > > This is similar to the proposed fix in java.base for > [JDK-8331879](https://bugs.openjdk.org/browse/JDK-8331879) / PR > openjdk/jdk#19130 (except for how I propose to handle the case of exactly > three slashes). > > As an alternative approach, I could fix just the 5 javadoc comments causing > the error, and leave the rest alone, but this will prevent problems from > cropping up in the future, and is in keeping with changes going into the JDK > source base. This pull request has now been integrated. Changeset: 31fe622c Author: Kevin Rushforth <k...@openjdk.org> URL: https://git.openjdk.org/jfx/commit/31fe622c4e540df06ed727343ace16cba3942534 Stats: 1309 lines in 50 files changed: 0 ins; 0 del; 1309 mod 8332732: Clean up non-standard use of /// comments in JavaFX Reviewed-by: angorya ------------- PR: https://git.openjdk.org/jfx/pull/1461
