Summary
`define-method` does not handle dotted argument lists
Metadata
- Id: a8079965411cd27f825194613c7d6293758276ec
- Trac id: 119
- Type: defect
- Reporter: felix
- Owner: tonysidaway
- Cc:
- Status: closed
- Component: extensions
- Estimated difficulty:
- Resolution: fixed
- Priority: minor
- Milestone:
- Version: 4.2.x
- Changetime: 2009-11-18 08:48:42 UTC
- Created: 2009-11-12 15:57:20 UTC
- Keywords: tinyclos
Description
`(define-method (foo ... . more) ...)` will throw an error. `#!rest` works fine, though.
Changes and comments
[2009-11-13 10:15:09 UTC] tonysidaway wrote:
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.
[2009-11-13 12:41:16 UTC] felix wrote:
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.
[2009-11-13 15:43:00 UTC] tonysidaway changed status from new to assigned
[2009-11-13 15:43:00 UTC] tonysidaway set owner to tonysidaway
[2009-11-13 15:43:00 UTC] tonysidaway wrote:
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.