Thread (3 messages) flat view 3 messages, 3 authors, 2025-09-17

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'.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help