Opened 5 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


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

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

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.