On Tue, 24 Mar 2026 19:29:44 GMT, Damon Nguyen <[email protected]> wrote:
> There are two instances within harfbuzz where the conditional checks seem > impossible or redundant. I have removed these redundant checks and tested the > changes. Everything still works as expected. > > The !last_range check is logically redundant because there is the other check > if (unlikely (last_range)) beforehand in the same segment. This is > effectively similar to a null check. > > The next check for peak != 0 is also redundant. This is because of if (peak > == 0 || coord == peak) earlier in the code that returns 1.f. So all following > instructions should logically be when peak is not 0. > > I have reported the issue in the open harfbuzz repo here: > https://github.com/harfbuzz/harfbuzz/issues/5873 > > The issue has been updated upstream. Now, this PR proposes to update it on > our end as well. Marked as reviewed by honkar (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/30412#pullrequestreview-4015934171
