Export to GitHub

wombat-ide - issue #233

loading miniKanren causes error in define.ss


Posted on Mar 29, 2013 by Quick Bird

A funny thing happened on the way to miniKanren:

~ (load "mk.scm")

Exception: invalid parameter list in (case-lambda ((h n n) (let (...) (...))) (others (error (...) "incorrect argument count"))) near line 22, char 9 of lib/wombat/define.ss

Attachments

Comment #1

Posted on Mar 29, 2013 by Helpful Elephant

I'm guessing that the original has something like this: (lambda (h n N) ...)

Wombat, by default is case-insensitive, so the two n's are the same. A few of the students in the class came across a similar error.

So it's the expected behavior, but the error message isn't very good. I should fix that.

(Also making it case-sensitive before loading miniKanren should also work.)

Comment #2

Posted on Mar 29, 2013 by Quick Bird

Yep. That was it! Maybe we should make Wombat case-sensitive by default?

Comment #3

Posted on Mar 29, 2013 by Helpful Elephant

I'm pretty sure that was a feature request way back in the beginning. I actually have to run a line when the Petite process starts up to force it to be insensitive.

Even if I fix it, I should make that error message better though.

Comment #4

Posted on Mar 30, 2013 by Quick Bird

Yes, I wanted to keep things clean and simple, and that's still appealing. It forces the use of ids like-this, not likeThis. However, it looks like case-sensitive is required to be #t with R6RS, so maybe we should go with the standard.

Comment #5

Posted on Aug 26, 2013 by Helpful Elephant

This issue was closed by revision 3d0894621987.

Status: Fixed

Labels:
Type-Defect Priority-Medium