Integrated: 8273910: Redundant condition and assignment in java.net.URI

Andrey Turbanov github.com+741251+turbanoff at openjdk.java.net
Thu Oct 7 16:59:11 UTC 2021


On Thu, 19 Aug 2021 19:08:59 GMT, Andrey Turbanov <github.com+741251+turbanoff at 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 <turbanoff at gmail.com>
Committer: Daniel Fuchs <dfuchs at 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


More information about the net-dev mailing list