On Wed, 20 Sep 2023 17:40:58 GMT, Phil Race <p...@openjdk.org> 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