Opened 15 years ago
Closed 15 years ago
#119 closed defect (fixed)
`define-method` does not handle dotted argument lists
Reported by: | felix winkelmann | Owned by: | Tony Sidaway |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | extensions | Version: | 4.2.x |
Keywords: | tinyclos | Cc: | |
Estimated difficulty: |
Description
(define-method (foo ... . more) ...)
will throw an error. #!rest
works fine, though.
Change History (4)
comment:1 follow-up: 2 Changed 15 years ago by
comment:2 Changed 15 years ago by
Replying to tonysidaway:
I'd like to look at this, with your permission, Felix. The innards of tinyclos are fresh in my mind and I can easily make define-method parse dotted argument lists. There may be ramifications in the compute-methods sequence but I'll watch out for them and fix them.
Yes, sure. Please go ahead.
comment:3 Changed 15 years ago by
Owner: | set to Tony Sidaway |
---|---|
Status: | new → assigned |
I've committed a fix and bumped the version to 1.6.2 (the egg is still marked hidden in the meta file).
There are some basic tests for methods with dotted arguments in tests/run.scm.
Please check this and close the ticket if it's satisfactory.
comment:4 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
I'd like to look at this, with your permission, Felix. The innards of tinyclos are fresh in my mind and I can easily make define-method parse dotted argument lists. There may be ramifications in the compute-methods sequence but I'll watch out for them and fix them.