vsedach/Parenscript

The function PARENSCRIPT::BODY is undefined.

Closed this issue ยท 9 comments

My code is broken since that commit: 8d7e8fa (I used git bisect to find which commit broke my code).

Here is the minimal code (that I found) to reproduce:

(ps
  (let* ((errorp "NIL"))))

P.S. I don't have this error with the Parenscript from quicklisp, but I have this error with Parenscript from github (master of this repo).

@agrostis You did this commit, maybe you could find the source faster than anyone else ๐Ÿ˜› ?

@agrostis You did this commit, maybe you could find the source faster than anyone else ๐Ÿ˜› ?

Yes, I'm afraid it's my fault. My commit made the occurence of the macro WITH-DECLARATION-EFFECTS in the special operator handler for SYMBOL-MACROLET precede the macro definition. I'll post a patch in a minute.

P.S. I don't have this error with the Parenscript from quicklisp, but I have this error with Parenscript from github (master of this repo).

Quicklisp uses the release version of Parenscript (v. 2.5). It might be time to release an updated version, but the decision is up to @vsedach.

Thanks! It broke my new and shiny feature, I was kind of sad ๐Ÿ˜ฅ
Can't wait for the fix :)

Ok, I will test it right away

Works
If I knew you were about to merge this I would've wait 2 minutes instead of adding your repo in my remotes ๐Ÿ˜›

8 min. ago it was only in my own fork. Please re-pull now.

Pulled from vsedach/Parenscript master. Worked like a charm, thanks again.

8 min. ago it was only in my own fork.

I know, that's what I was saying by "adding your repo in my remotes". I added your fork to my remotes and tested with the branch "devel".