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; This pull request has now been integrated. Changeset: 7de2cf85 Author: Andrey Turbanov <turban...@gmail.com> Committer: Daniel Fuchs <dfu...@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/7de2cf852d75ea6eb039e69067d4e32421283de5 Stats: 11 lines in 1 file changed: 4 ins; 7 del; 0 mod 8273910: Redundant condition and assignment in java.net.URI Reviewed-by: dfuchs ------------- PR: https://git.openjdk.java.net/jdk/pull/5190
