On Sun, Jan 17, 2021 at 12:10 AM Thien-Thi Nguyen <firstname.lastname@example.org> wrote:
> your timing is fortuitous. I just spent the christmas
> holidays delving into GUILE 1.8's heap expansion strategy and
> found and fixed a bug with it.
> How does code review for proposed patches work these days?
> I don't know. I think what you did is fine. I invite experts
> to review and comment. They will surely be quicker than me (i
> will require a week or two just to read/understand the patch).
> I'd be happy to polish it up (ie. add a proper test) for
> inclusion in GUILE 1.8.9,
> Sounds good. I think the general approach for 1.8.x releases
> will be bugfixes and documentation changes primarily, so your
> change would be most welcome (once i wrap my head around it).
Thanks. It turns out my previous fix introduced ABI breakage, so I
reworked it to not change function signatures or struct sizes. It's
also split up in more parts, so it becomes easier to understand.
Please see here:
Any news here? Can I do anything to get this fix in?