bug#57948: [PATCH] Avoid 'frame-local-ref' errors when printing backtrace.

2023-01-11 Thread Andrew Whatson
Ludovic Courtès wrote: > > Applied, thanks! Thank you! > PS: Please use ‘git format-patch’ so the patch can be directly consumed > by ‘git am’. My mistake, sorry about that. Noted for next time :) Cheers, Andrew

bug#57948: [PATCH] Avoid 'frame-local-ref' errors when printing backtrace.

2023-01-11 Thread Ludovic Courtès
Hi, Andrew Whatson skribis: > commit 164bdce6acf53796cb96ef1930a89c6caf84bc39 > Author: Andrew Whatson > Date: Wed Jan 11 14:04:32 2023 +1000 > > Test for 'frame-local-ref' errors when printing backtrace. > > This test reproduces the error from , and >

bug#57948: [PATCH] Avoid 'frame-local-ref' errors when printing backtrace.

2023-01-10 Thread Andrew Whatson
Ludovic Courtès wrote: > > It would be great if you could add a simple test case though, so that > the bug doesn’t eventually come back to haunt us. I've finally tracked this one down, a patch with a unit test for this bug is attached. Cheers, Andrew commit 164bdce6acf53796cb96ef1930a89c6caf84bc

bug#57948: [PATCH] Avoid 'frame-local-ref' errors when printing backtrace.

2022-10-13 Thread Ludovic Courtès
Hi, Andrew Whatson skribis: > Ludovic Courtès wrote: >> >> It would be great if you could add a simple test case though, so that >> the bug doesn’t eventually come back to haunt us. >> >> Could you send an updated patch? > > Ah yes, getting this covered in a test is on my list. I had trouble >

bug#57948: [PATCH] Avoid 'frame-local-ref' errors when printing backtrace.

2022-10-12 Thread Andrew Whatson
Ludovic Courtès wrote: > > It would be great if you could add a simple test case though, so that > the bug doesn’t eventually come back to haunt us. > > Could you send an updated patch? Ah yes, getting this covered in a test is on my list. I had trouble writing a reproducer previously, I think t

bug#57948: [PATCH] Avoid 'frame-local-ref' errors when printing backtrace.

2022-10-12 Thread Ludovic Courtès
Hi Andrew, Andrew Whatson skribis: > Workaround for . > > * module/system/vm/frame.scm (frame-call-representation): Treat a > binding as "unspecified" if its slot exceeds 'frame-num-locals'. Yay, great to see that fixed (or almost)! It would be great if you could ad