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
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
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;
-
Commit messages:
- [PATCH] Cleanup redundant assignment and condition in URI
