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 |
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
| Milestone: | someday → 5.1 |
|---|
comment:2 Changed 5 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Should be fixed with the changes pushed with from #1581.
Moving to 5.1, as it's related to #1581