On Wed, 3 Sep 2025 13:57:41 GMT, Daniel Gredler <dgred...@openjdk.org> wrote:
>> src/java.desktop/share/classes/java/awt/font/TextLayout.java line 516: >> >>> 514: >>> 515: int start = text.getBeginIndex(); >>> 516: int limit = text.getEndIndex(); >> >> Why do we call text.first() twice, on line 516 and in the loop init block? > > I'm not sure, I don't think it's necessary. The code has been this way since > the first public commit, so there is no hint in the public git history. > Should I remove it? Looks to me like it can be safely removed (the one on line 516/518) ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/26947#discussion_r2319714886
