Subject: Re: [PATCH v3 10/15] replay: make it a minimal server side command

From: Toon Claes <[email-masked]>
Date: Thu, 22 Jun 2023 12:01:11 +0200

Christian Couder <[email-masked]> writes:

> +			decoration = decoration->next;
> +		}
>  	}
>
> +	/* Cleanup */

Nit: I don't think this comment adds much value. I would argue it's more
confusing to have it, because there's a label cleanup: just a few lines
down.

>  	merge_finalize(&merge_opt, &result);
> +	ret = result.clean;