On Sun, Jan 17, 2021 at 12:10 AM Thien-Thi Nguyen <ttn@gnuvola.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.
>
> https://github.com/hanwen/guile/commit/9b32504780e0b604196be866b8c36079891e3cd6
>
> 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:
https://gcithub.com/hanwen/guile/commit/8fbe3222cac4b4e9b39a6a3570ac43f160faa516
Any news here? Can I do anything to get this fix in?
--