gilch / hissp

It's Python with a Lissp.
https://gitter.im/hissp-lang/community
Apache License 2.0
364 stars 9 forks source link

Synexpand fixes #223

Closed gilch closed 1 year ago

gilch commented 1 year ago

Found and fixed some issues. Still not tested well enough. _rewrite isn't even used as macro anymore. Expect some instability.

gilch commented 1 year ago

Ha. The direct recursion tallying feature from #220 no longer has any usages in any examples, so I no longer have any test coverage for that branch. The examples using it were all in the synexpand docstring. I know the current version works and I also want to keep the feature. I should probably demonstrate recursive macros in the docs somewhere. I'm going to merge this anyway, even though that means giving up my 100% coverage. Probably temporarily. I also kind of doubt my macros are 100% covered, or those bugs I just fixed would have already been caught. I'm not sure how useful a coverage report on the compiled Python would be though. I'll have to try it.