On Fri, Jul 21, 2017 at 5:16 PM, Marc Nieper-Wißkirchen <xxxxxx@nieper-wisskirchen.de> wrote:

We should also ask Alex Shinn to bring his implementation in line with the other (and previous) implementation. There is a simple change necessary here:


Just change "compare" into "eq?", which is bound-identifier=? in the syntactic closures implementation, Chibi is using. (I made the change locally, and all tests continue to pass.)

So, to simplify, we're assuming the syntax-rules literals and patterns are
always written together, and never "assembled" from separate expansions.
Thus bound-identifier=? is used (although free-identifier=? is still used for
matching the literal to an expression).

I can think of counter-examples, but none especially compelling, and am
convinced by Al's example that bound=? makes more sense.  I've pushed
the change to master.

-- 
Alex