[Tarantool-patches] [PATCH 4/4] txn_limbo: simplify txn_limbo_on_parameters_change

[Serge Petrenko]

22.04.2021 14:34, Cyrill Gorcunov пишет:
> There is no need to test @confirm_lsn since we're
> continue the list traversing in any case.
>
> I keep "continue" here to be consistent with other "if"s.
>
> Signed-off-by: Cyrill Gorcunov <gorcunov at gmail.com>
> ---
>   src/box/txn_limbo.c | 3 +--
>   1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/src/box/txn_limbo.c b/src/box/txn_limbo.c
> index 0ac8fff25..c07db3025 100644
> --- a/src/box/txn_limbo.c
> +++ b/src/box/txn_limbo.c
> @@ -707,8 +707,7 @@ txn_limbo_on_parameters_change(struct txn_limbo *limbo)
>   		assert(e->ack_count <= VCLOCK_MAX);
>   		if (!txn_has_flag(e->txn, TXN_WAIT_ACK)) {
>   			assert(e->lsn == -1);
> -			if (confirm_lsn == -1)
> -				continue;
> +			continue;
>   		} else if (e->ack_count < replication_synchro_quorum) {
>   			continue;
>   		} else {
LGTM.

-- 
Serge Petrenko