Re: RFR: 8273910: Redundant condition and assignment in java.net.URI

2021-10-06 Thread Daniel Fuchs
On Thu, 19 Aug 2021 19:08:59 GMT, Andrey Turbanov 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

Re: RFR: 8273910: Redundant condition and assignment in java.net.URI

2021-10-06 Thread Daniel Fuchs
On Thu, 19 Aug 2021 19:08:59 GMT, Andrey Turbanov 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; I have sent this patch