[PATCH 1/1] Fix typo'd procedure name in "emit-import-library" declaration handling

[Evan Hanson]

This is intended to check for a pair of strings when this declaration is
used in its `(MODULENAME FILENAME)' form, but we currently use `string'
instead of `string?' to check the second item, which will always trigger
an error ("bad argument type").
---
 core.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core.scm b/core.scm
index e548bbc0..9e5a8c62 100644
--- a/core.scm
+++ b/core.scm
@@ -1717,7 +1717,7 @@
  (cond ((symbol? il)
 (cons il (string-append (symbol->string il) 
".import.scm")) )
((and (list? il) (= 2 (length il))
- (symbol? (car il)) (string (cadr il)))
+ (symbol? (car il)) (string? (cadr il)))
 (cons (car il) (cadr il)))
(else
 (warning
-- 
2.29.3

[PATCH 0/1] Fix typo'd procedure name in "emit-import-library" declaration handling

[Evan Hanson]

Hi there,

This is just the fix that was included in the earlier declarations
changes, which megane rightly dropped from that patch since it wasn't
really related. It makes this form of the "emit-import-library"
declaration work again:

  (declare (emit-import-library (MODULE FILENAME)))

Cheers,

Evan

Re: [PATCH 0/1] Add `emit-types-file` declaration

[megane]

Evan Hanson  writes:

> Hi there,
>
> This small patch closes #1644.
>
> Thanks to Marco Maggi for suggesting this feature in a chicken-users
> thread from a while back.

Pushed. Thanks for you both.

There was a fix for apparent typo in argument handling for emit-impor-library
declaration. I dropped it for sake of more surgigal commit.

I also added a bit of documentation in manual/Types.

>
> Cheers,
>
> Evan