RFR: 8336332: Rework tests to avoid unrelated stderr output [v9]

Andy Goryachev angorya at openjdk.org
Thu Sep 18 18:04:25 UTC 2025


On Thu, 18 Sep 2025 17:48:29 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Andy Goryachev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   static
>
> modules/javafx.base/src/test/java/test/javafx/util/OutputRedirect.java line 161:
> 
>> 159:             map((s) -> findException(s)).
>> 160:             forEach((c) -> {
>> 161:                 if (c != null) {
> 
> Minor suggestion: filter for non-null Strings before the `forEach` rather than checking for non-null in the `forEach` ?

my version is _more efficient_ :-)

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1897#discussion_r2360552981


More information about the openjfx-dev mailing list