On Tue, 24 Sep 2024 16:12:29 GMT, Alexey Ivanov <[email protected]> wrote:
>> I added description to all the constructors available in `PassFailJFrame`. >> >> I generated the javadoc for it: [`PassFailJFrame` constructor >> details](https://cr.openjdk.org/~aivanov/PassFailJFrame/api/PassFailJFrame.html#constructor-detail) >> which is easier to read. This hosted javadoc could be used as a reference >> to its API. >> >> The constructor with the largest number of parameters contains a short >> description of how to use the `PassFailJFrame` object. >> >> Shall we deprecate the constructors in favour of `Builder`? >> >> The builder provides a more streamlined experience, therefore new tests >> should use it instead of constructors. We may update the tests which use the >> constructors, after which the constructors can be removed. > > Alexey Ivanov has updated the pull request incrementally with one additional > commit since the last revision: > > Use EDT abbreviation in constructor description > > The term EDT (Event Dispatch Thread) is introduced in > the top-level doc comment. Marked as reviewed by honkar (Reviewer). Doc changes looks good. ------------- PR Review: https://git.openjdk.org/jdk/pull/21145#pullrequestreview-2326708134 Marked as reviewed by honkar (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/21145#pullrequestreview-2326708441
