On Tue, 29 Apr 2025 12:42:28 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

>> test/jdk/java/awt/regtesthelpers/PassFailJFrame.java line 1306:
>> 
>>> 1304:      * <p>A sample usage can be found in
>>> 1305:      * <a 
>>> href="https://github.com/openjdk/jdk/blob/7283c8b/test/jdk/java/awt/FileDialog/SaveFileNameOverrideTest.java#L84";>{@code
>>> 1306:      * SaveFileNameOverrideTest.java}</a>
>> 
>> Javadoc update LGTM. Maybe the doc for forceFail() can be updated similarly?
>> @aivanov-jdk What do you suggest?
>> 
>> It can be done as a part of this PR change or a separate PR. Since it is a 
>> similar doc change, I think it can be done as part of this PR by the 
>> changing the JBS title appropriately  - _Update javadocs for 
>> PassFailJFrame's forcePass() & forceFail() methods_
>
> @honkar-jdk JBS title could be updated… I'm still against doing so.
> 
> Currently, `forcePass` provides an example that shouldn't be followed. It's a 
> problem.
> 
> Expanding the docs for `forcePass` and `forceFail` would be a good addition, 
> but it's not such a big issue. I've submitted 
> [JDK-8355515](https://bugs.openjdk.org/browse/JDK-8355515) to track this work.
> 
> As I mentioned in the description of JDK-8355515, adding a section in the 
> main javadoc of `PassFailJFrame` would be even better. Although a new section 
> doesn't eliminate expanding documentation for the methods themselves.

Sounds good to me if forceFail() docs is going to be handled as separate issue.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/24837#discussion_r2066927589

Reply via email to