On Wed, 20 Sep 2023 17:40:58 GMT, Phil Race <[email protected]> wrote:
>> Karthik P K has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Review comments
>
> modules/javafx.graphics/src/main/java/com/sun/javafx/text/PrismTextLayout.java
> line 760:
>
>> 758: int lineCount = getLineCount();
>> 759: while (index < lineCount) {
>> 760: if (text != null) {
>
> why aren't you using "textFound" as in if "(!textFound) {"
Updated code to use `textFound`
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1157#discussion_r1343838434
