On Tue, 29 Apr 2025 11:45:17 GMT, Manukumar V S <m...@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 @aivanov-jdk Can I do it as part of 
> [JDK-8355515](https://bugs.openjdk.org/browse/JDK-8355515)?.

Absolutely!

I don't want to include any changes for `forceFail` here because they're 
unrelated to the problem we're addressing.

[The description](https://bugs.openjdk.org/browse/JDK-8355515#description) of 
JDK-8355515 explicitly mentions the intent:

> The description of `forceFail()` should be expanded, too.

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

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

Reply via email to