Arc Forumnew | comments | leaders | submitlogin
1 point by akkartik 4857 days ago | link | parent

Hmm, (aif t nil) works fine for me. aif perhaps doesn't need that gensym even if delisting does in general.


1 point by akkartik 4851 days ago | link

Ack, I misunderstood you. I've retreated to a version with car/cdr manipulation; looks like there's no way around it (it's still simpler than the original, but that just makes me wonder what bugs remain)

https://github.com/akkartik/wart/commit/9425c4829abb88052d29...

-----

1 point by rocketnia 4851 days ago | link

I haven't tested it, but I don't see any bugs from here.

I'm a bit surprised you broke up the (if branches ...) condition into two places like that, though. I think it would have been simpler just to replace "(or then more-branches)" with "branches". ^^

-----

1 point by akkartik 4851 days ago | link

:) Yeah I thought of that in the next CL.

-----