Re: [PATCH v2 03/10] xdiff: make xrecord_t.ptr a uint8_t instead of char

From: Ezekiel Newren <ezekielnewren@gmail.com>
Date: Nov 6, 2025, 23:14 UTC
Message-ID: <CAH=ZcbDTnvgrkfKYe_uyPHh5Xd2Pbw4532UvNcz+nz+rpbhUiQ@mail.gmail.com>
In-Reply-To: <3f7bbb5e-0d67-4ef2-82fb-e0b00683c178@gmail.com>

On Thu, Nov 6, 2025 at 3:55 AM Phillip Wood <phillip.wood123@gmail.com> wrote:

Show 12 quoted lines

>
> On 29/10/2025 22:19, Ezekiel Newren via GitGitGadget wrote:
> > @@ -156,8 +156,8 @@ static int xdl_prepare_ctx(unsigned int pass, mmfile_t *mf, long narec, xpparam_
> >                       if (XDL_ALLOC_GROW(xdf->recs, xdf->nrec + 1, narec))
> >                               goto abort;
> >                       crec = &xdf->recs[xdf->nrec++];
> > -                     crec->ptr = prev;
> > -                     crec->size = (long) (cur - prev);
> > +                     crec->ptr = (uint8_t const *)prev;
> > +                     crec->size =(long) ( cur - prev);
>
> The changes to crec->size here look unintentional

I agree. I'll change that.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.