Skip to content

Fix declaration syntax error#151

Open
vibs29 wants to merge 1 commit into
edicl:masterfrom
vibs29:syntax_error
Open

Fix declaration syntax error#151
vibs29 wants to merge 1 commit into
edicl:masterfrom
vibs29:syntax_error

Conversation

@vibs29

@vibs29 vibs29 commented Jan 10, 2026

Copy link
Copy Markdown

No description provided.

@stassats

Copy link
Copy Markdown
Member

How's that a syntax error?

@hanshuebner

Copy link
Copy Markdown
Member

How's that a syntax error?

Some ancient lisps object. Not sure which one this is meant to appease.

@vibs29

vibs29 commented Jan 10, 2026

Copy link
Copy Markdown
Author

Oh you're right, it's not a syntax error. Sorry. Please reject this pull request.

I realise now that "type" is optional in (declare (type ...)).

When I loaded drakma into clisp (a recent 2024 version), this message went by as it compiled encoding.lisp:

WARNING: in DECODE-FLEXI-STREAM in lines 89..107 : Unknown declaration
         FLEXI-INPUT-STREAM.
         The whole declaration will be ignored.

so I took that to mean a missing "type" and added it, which indeed got rid of the warning. But I don't know why, because I've checked that it accepts (declare (fixnum x)) without a "type".

@hanshuebner

Copy link
Copy Markdown
Member

@vibs29 That begs the question why you try to use CLISP in the first place. It is unmaintained and - as you've noted already - has quite some quirks and deficiencies that make it difficult to get along with if you try using common open source libraries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants