Re: [PATCH 00/17] Use rust types in xdiff.
From: Elijah Newren <hidden>
Date: 2025-09-17 02:16:40
On Tue, Sep 16, 2025 at 3:01 PM Ezekiel Newren [off-list ref] wrote:
On Tue, Sep 16, 2025 at 3:56 PM Junio C Hamano [off-list ref] wrote:quoted
I think we saw that the earlier part were read carefully by Elijah (and others may have read without finding anything worth commenting on), so should we split this into two parts and start merging the early 9 down to 'next' and then to 'master'?I agree. 1-9 are ready to go. Do I need to create a new version of this patch series? let it stand as is until it's been merged into master, or something else?
I think 1-9 are close to ready to go, but there's several small cleanups that would be nice to have in a v2 on patches 2, 4, 7, 8, 9. See my comments on the patches, but it's things like adding detail to commit messages or otherwise touching those up, removing orthogonal style cleanups (or making them a separate patch), and removing extra blank lines. Could we get a re-roll of just the first 9 patches with these addressed? Then I think it'd be ready to merge down to 'next'.