Skip to content

Fix lambda macro having parameters in wrong namespace#80

Open
stag-enterprises wants to merge 1 commit intovseloved:masterfrom
stag-enterprises:patch-1
Open

Fix lambda macro having parameters in wrong namespace#80
stag-enterprises wants to merge 1 commit intovseloved:masterfrom
stag-enterprises:patch-1

Conversation

@stag-enterprises
Copy link

@stag-enterprises stag-enterprises commented Dec 10, 2024

When using the ^() macro, it sometimes declares the argument variables into the rutils.readtable package, instead of the current package. For example:

(macroexpand '^(+ % %%))
; produces
#'(LAMBDA (&OPTIONAL RUTILS.READTABLE:% RUTILS.READTABLE:%%)
  (DECLARE (IGNORABLE RUTILS.READTABLE:%) (IGNORABLE RUTILS.READTABLE:%%))
  (+ % %%))

As you can see, the % and %% variables used in + are not the same as the ones defined as a parameter, so this will error. The solution is to use INTERN, so that the % and %% variables are declared in the correct package (the current one in the evaluation of the macro).

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.

1 participant