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