On Thu, May 21, 2020 at 03:12:33PM -0500, Jim Ursetto wrote: > Hey Peter,
Hi Jim, Good to see you're back! > I might be misunderstanding, but quite a few Chicken 5 eggs still use > define-record-printer, and it seems to work fine in Chicken 5.1.0 and 5.2.0. hm, maybe it's just the way we used it in core that's wrong? I dunno. I can't really explain it because the expansion looks fine and is the same as the one in sql-de-lite. Anyway, good to know that it's not completely broken. > I’d be amenable to replacing this with set-record-printer! in my eggs for > Chicken 5 (I was unaware this was deprecated), but you’d have to hunt down a > lot of references if you want to remove it. We can keep a long deprecation period. This patch just removes usages in core and updates the macro so we can get rid of the obsolete alias. Cheers, Peter
signature.asc
Description: PGP signature
