On Thu, 18 Sep 2025 17:48:29 GMT, Kevin Rushforth <[email protected]> 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
