Re: [PATCH] rust: alloc: add doc test for `Vec::from_elem`
From: Alexandre Courbot
Date: Sun Apr 26 2026 - 09:11:25 EST
On Sun Apr 26, 2026 at 1:27 PM JST, Hsiu Che Yu wrote:
> On Sun, Apr 26, 2026 at 01:05:36PM +0900, Alexandre Courbot wrote:
>>Received just as I had finished testing your other series. :) So while
>>I'm at it:
>
> On Sun, Apr 26, 2026 at 01:00:06PM +0900, Alexandre Courbot wrote:
>>Looking good - just one thing, when sending a new revision of a
>>patchset, please start a new thread instead of replying to the previous
>>one. Otherwise things can get pretty hard to follow once we arrive at,
>>say, v10. :)
>
> I just realized — should I have grouped the tests targeting the same
> module into a single patch series (like the one that was split into
> two recently)?
> Would that make the review process easier? If so, I'll start batching
> small doc-test patches together before sending.
I don't think there is any hard rule, but usually a patch series implies
a dependency between the patches, i.e. patch 2 cannot build without
patch 1. For what you sent so far this is not the case, so separate
patches is fine and maybe even preferable as they can be applied
individually whereas maintainers tend to consider series as "all or
nothing" (there are exceptions of course).
>
> Also, as you pointed out in your other reply,
> I had confused `link:` and `in-reply-to`.
> I'll make sure to use `link:` to reference context for v2 onwards.
No worries. If I may recommend b4 [1] for managing your patch series, it
ususally just does the right thing for you.
[1] https://b4.docs.kernel.org/en/latest/