On Thu, 19 Aug 2021 19:08:59 GMT, Andrey Turbanov <github.com+741251+turban...@openjdk.org> wrote:
> 1. Assignment `ru.host = child.host;` is duplicated and hence redundant. > 2. Condition `q > p` is always `true`, because it just bellow inverse check > > if (q <= p) > break; The updated logic looks correct, and my tests (tier1,tier2,tier3) came back with no failure. Reviewed. ------------- Marked as reviewed by dfuchs (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/5190
