Re: [PATCH] Quit compiling when an invalid import-Library declaration is encountered

2021-04-22 Thread Evan Hanson
Imagine that last email had a nice descriptive subject, like this one. Evan

[no subject]

2021-04-22 Thread Evan Hanson
Thanks megane! On 2021-04-22 14:00, megane wrote: > ITYM it'll never trigger an error, even if the the filname is symbol for > example. In the use case where FILENAME is a string you will hit this: $ csc -analyze-only - < to be added to the alist of import libraries, leading to an error

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

2021-04-22 Thread megane
Evan Hanson writes: Thanks, pushed. > 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").

Re: FIXED [PATCH] Report more information for unresolved identifiers in modules

2021-04-22 Thread megane
Evan Hanson writes: > Cool, here is a signed off copy. But, I made some small changes, so if you > or another hacker could have a look just to make sure you're OK with > it? Looks good, pushed! > > I fixed two places where the `resolve-variable` procedure didn't have the new > `outer-ln`