Summary

Update explicit info strings in calls to `getter-with-setter`

Metadata

• Id: 306bc7aa599e8ea005b5b4d866bf0590c9e3d5d9
• Trac id: 1361
• Type: enhancement
• Reporter: felix
• Owner:
• Cc:
• Status: closed
• Component: core libraries
• Estimated difficulty: trivial
• Resolution: fixed
• Priority: minor
• Milestone: 5.0
• Version: 5.0.0
• Changetime: 2018-05-21 12:45:51 UTC
• Created: 2017-04-09 00:06:51 UTC
• Keywords:

Description

`getter-with-setter` allows passing an optional info string that is shown when printing a procedure. Currently these strings do not contain the module prefix, which is added for internally generated info strings.

For consistency, we should add the module-prefixes manually in uses of `getter-with-setter`.

Changes and comments

[2017-04-19 11:03:12 UTC] kristianlm wrote:

I think it's possible that if we fix #1363, we can get away in some cases by removing the info-string altogether. Take this, for example:

 ;; library.scm:3045
 (set! caar (getter-with-setter caar (lambda (x y) (set-car! (car x) y)) "(caar p)"))

From what I gather, the only reason we can't just re-use the info-string from the original caar getter is issue #1363.

[2018-05-21 12:45:51 UTC] sjamaan changed status from new to closed

[2018-05-21 12:45:51 UTC] sjamaan set resolution to fixed

[2018-05-21 12:45:51 UTC] sjamaan wrote:

This should be alright now.