[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Export procedures from srfi-17
From: |
Peter Bex |
Subject: |
Re: [PATCH] Export procedures from srfi-17 |
Date: |
Thu, 19 Aug 2021 14:20:06 +0200 |
On Thu, Aug 19, 2021 at 01:58:48PM +0200, Peter Bex wrote:
> PS: aren't all the register-primitive-module invocations for srfis wrong,
> in that they register a module under a similarly named unit, but there
> is no such unit library (in any of these cases)?
I thought about it some more and realised that the "module-library" field
of the module record type is only used to emit (##core#require ...) in
chicken.syntax#import-definition, and ##core#require treats
"builtin-features" as being already linked, which includes the srfis.
It also treats "core-units" as being already linked, which includes
"library" among others, of course.
I think we can simplify this logic by dropping the builtin-features bit,
as that seems like it's a hack specific for SRFIs. I've filed
https://bugs.call-cc.org/ticket/1777 for this. Of course not something
we should do for 5.3.
Cheers,
Peter
signature.asc
Description: PGP signature