Clojurians Log v2

@jeroenvandijk I think I may have a nice solution for the macro rewriting problem: https://github.com/borkdude/sci/issues/397

yeah I think that would work and be user friendly when using the backtick. The only thing would be that the namespace symbol of the implementation would end up in Sci context somewhere. Not sure if that is a real issue

Why would that be an issue?

maybe if you would like to hide the underlying namespace structure to the Sci user

thinking out loud here, not even sure what you could do with that

If you would want to do that, then you could just not use this :)

good point

Sounds like a good solution

The macros that will work with this are still only those who don't cause side effects at compile time and probably also not use &env

But that's most of the macros anyway

Another (more costly, but maybe not significant) solution would be to make sci postwalk the macroexpansion and replace the external namespaces with inner ones

Then the macro-expansion will also look more pretty

In a repl environment and using macroexpand the last option would be the least surprising as it doesn’t leak implementation details. But the performance concern is also a valid one

Maybe it's not that significant though, since this happens only once, not in a loop or anything

would one or the other matter for the output of stacktraces?

Good point. Maybe, I have to test this. I'll write it down in the issue.