Opened 6 years ago

Closed 5 years ago

#1583 closed defect (fixed)

coops record-printer for instances do not work

Reported by: Kooda Owned by:
Priority: minor Milestone: 5.1
Component: extensions Version: 5.0.0
Keywords: Cc:
Estimated difficulty: easy

Description

As reported on IRC by matijja, coops’ record printer for class instances does not work.

http://paste.call-cc.org/paste?id=c0e12d0a9d9bbd2e513fe88092f38be9e7afb6c3

While redefining a record printer for it makes everything work as expected:

http://paste.call-cc.org/paste?id=4e21716d788acd4c207a5a1ee14e11fa8cfd952e

From what I could gather, coops uses the record-variant egg to define its coops-instance record type.

It appears the egg has not been updated to add the new module prefix of the record symbol, so it registers the record type as just being coops-instance, while the define-record-printer macro correctly adds the module prefix, making them distinct from each other, the printer is then never used.

If we update the record-variant egg, we also would have to update coops, because it uses ##sys#check-structure directly.

Change History (2)

comment:1 Changed 5 years ago by sjamaan

Milestone: someday5.1

Moving to 5.1, as it's related to #1581

comment:2 Changed 5 years ago by felix winkelmann

Resolution: fixed
Status: newclosed

Should be fixed with the changes pushed with from #1581.

Note: See TracTickets for help on using tickets.