#91 ,complete command is finicky 3 days ago

Comment by ~technomancy on ~technomancy/fennel

I ended up not merging this branch; this problem does not seem to manifest on the original implementation.

REPORTED RESOLVED FIXED

#85 locals are lost in the repl after 2b4b31c6 3 days ago

Comment by ~technomancy on ~technomancy/fennel

I think 20e6b50 fixes this.

REPORTED RESOLVED FIXED

#85 locals are lost in the repl after 2b4b31c6 4 days ago

Comment by ~technomancy on ~technomancy/fennel

Ahhhh this approach cannot work because it wraps the input code in a let. We can work around this for locals by using debug.getlocal to splice things back into the top level env but when there's an inline macro defined there it won't work at all; the macro is defined in the let scope and there's no way to splice it back into the top-level scope.

#81 if returns zero values in case false branch is not specified 15 days ago

Comment by ~technomancy on ~technomancy/fennel

Thanks; went ahead and merged it. Will try to get some more eyes on this change as it's rather large.

REPORTED RESOLVED FIXED

#91 ,complete command is finicky 20 days ago

Comment by ~technomancy on ~technomancy/fennel

I see; thanks for the clarification. So in this case the repl can successfully see the locals; it's just the completion doesn't see them? And it happens to the same kind of modules as the other bug?

#90 Warning generated when referencing globals from macros, despite options 20 days ago

Comment by ~technomancy on ~technomancy/fennel

Are you just talking about the fact that the gensym local name numbers keep growing rather than starting back over at zero for every new scope? That's an older change from 0.10.0 so it shouldn't have anything to do with the changes discussed here.

#91 ,complete command is finicky 20 days ago

Comment by ~technomancy on ~technomancy/fennel

This is very likely to be the same problem as #85. I haven't merged that fix to main yet. Do you have this problem on the new-locals-saving branch?

#40 Provide a way to compile macros to Lua to ship those with luarocks 21 days ago

Comment by ~technomancy on ~technomancy/fennel

This has worked for a while now, but I added a test for it: https://git.sr.ht/~technomancy/fennel/commit/6e88b217bfc493a7d5448fb884e50cf7717643e3

REPORTED RESOLVED FIXED

#83 (values) gives weird results on boolean operators 21 days ago

Comment by ~technomancy on ~technomancy/fennel

Fixed in https://git.sr.ht/~technomancy/fennel/commit/3da46828ca548797300e57e928185f6815fe7956 ; thanks for pointing this out.

REPORTED RESOLVED FIXED

#83 (values) gives weird results on boolean operators 21 days ago

Comment by ~technomancy on ~technomancy/fennel

Oh, yes, you're right. I was only considering the tail position. Mid-arglist there's nothing wrong with collapsing things down to one, and it should be easy to implement.