[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Fix #1757 by not merging module environment into the syntact
From: |
Evan Hanson |
Subject: |
Re: [PATCH] Fix #1757 by not merging module environment into the syntactic environments of reexported macros |
Date: |
Sat, 31 Jul 2021 16:05:52 +1200 |
On 2021-07-14 12:45, Peter Bex wrote:
> Here's a reasonably straightforward patch (I hope) for #1757.
> Commit should speak for itself, even if the stuff itself is
> a bit tricky.
It really is, this took some time to understand... But it does seem like
the right thing, now that you've spelled it out.
I was wondering whether there are any implications in changing the order
of the `sexports` field of the module, now that we're putting all
"normal" syntax exports before any re-exported ones. However, after a
bit of digging, I don't think this should matter?
Anyway, applied. Thank you Peter!
Evan