Re: [Chicken-hackers] [PATCH 0/1] Add "chicken.module" module

[Peter Bex]

On Sat, Jun 17, 2017 at 12:05:35PM +1200, Evan Hanson wrote:
> Hi all,
> 
> Here's a patch that adds a "chicken.module" module according to some of
> the discussion in the "Add unexport form to modules" thread, and from
> the "undecided" section of c-l-r (which already proposed a lot of the
> same changes, in fact).
> 
> The library is syntax-only and contains CHICKEN's "module language"
> (well, most of it -- it doesn't include `functor` or `define-interface`
> quite yet). It's actually a pretty small change, mostly just reshuffling
> things in expand and capturing the macro environment at the right time.

Looks good to me, pushed!  I've moved (chicken module) up to "decided"
and added a few missing import variants to the list of identifiers.
What's the reason you've only marked functor and define-interface as
TODO and not moved them yet?  Are they problematic somehow?

Cheers,
Peter

signature.asc
Description: PGP signature