Re: [Chicken-hackers] [PATCH] * eval.scm (compile): Remove failing environment lookup in ##core#set!

2018-08-04 Thread Peter Bex
On Fri, Aug 03, 2018 at 09:39:46PM +0300, megane wrote: > > I couldn't come up with a way to test this. You > can test the old one if you add print to the case and run this: > > (let ([l '(##core#set! x 1)]) > (##sys#current-environment (cons `(,l . foo) (##sys#current-environment))) > (eval

Re: [Chicken-hackers] [PATCH] * eval.scm (compile): Remove failing environment lookup in ##core#set!

2018-08-04 Thread felix . winkelmann
Pushed. felix ___ Chicken-hackers mailing list Chicken-hackers@nongnu.org https://lists.nongnu.org/mailman/listinfo/chicken-hackers

[Chicken-hackers] [PATCH] Fix crash from #1493

2018-08-04 Thread Peter Bex
Hi all, Megane found an interesting case where the macro-expander would crash hard (at least in a debugbuild) when a macro expands to a macro definition for an identifier that's already bound to a macro. After some consideration I believe megane's patch is correct, so attached is that fix, plus

Re: [Chicken-hackers] [PATCH] Fix crash from #1493

2018-08-04 Thread Evan Hanson
Looks good, applied to both branches. Thanks both, Evan ___ Chicken-hackers mailing list Chicken-hackers@nongnu.org https://lists.nongnu.org/mailman/listinfo/chicken-hackers