[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Chicken-hackers] [PATCH 0/1] Add "chicken.module" module
From: |
Peter Bex |
Subject: |
Re: [Chicken-hackers] [PATCH 0/1] Add "chicken.module" module |
Date: |
Sun, 18 Jun 2017 15:08:06 +0200 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
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