From: Toon Claes <toon@iotcl.com>
Date: Thu, 22 Jun 2023 10:01:11 GMT
Subject: Re: [PATCH v3 10/15] replay: make it a minimal server side command
Message-ID: <87pm5nstli.fsf@iotcl.com>
In-Reply-To: <20230602102533.876905-11-christian.couder@gmail.com>


Christian Couder <christian.couder@gmail.com> 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;