Opened 3 years ago
Closed 3 months ago
#1780 closed defect (wontfix)
wishlist: support keywords in coops (make <class> ...) form
Reported by: | Shawn Rutledge | Owned by: | felix winkelmann |
---|---|---|---|
Priority: | major | Milestone: | someday |
Component: | extensions | Version: | 5.2.0 |
Keywords: | Cc: | ||
Estimated difficulty: |
Description
As described in SRFI-88, keywords are nice to have for widget constructors:
(tcltk-frame relief: 'ridge
borderwidth: 4
height: "50px"
width: "100px")
but if widgets are coops classes, it's natural to use the make
form rather than a separate constructor procedure for each class:
(make <widget> width: 100 height: 50 ...)
I got it working with a quick hack (which no doubt could be more elegant):
Index: coops.scm =================================================================== --- coops.scm (revision 40393) +++ coops.scm (working copy) @@ -9,9 +9,9 @@ (declare (disable-interrupts)) -(import scheme matchable srfi-1 record-variants +(import scheme matchable srfi-1 record-variants symbol-utils (chicken condition) (chicken fixnum) (chicken format) - (chicken syntax) (chicken sort) + (chicken keyword) (chicken syntax) (chicken sort) (only miscmacros ensure)) (import-for-syntax matchable (chicken plist) srfi-1) @@ -176,6 +176,7 @@ (sv (make-vector (length slotnames) uninitialized)) (i (make-coops-instance c sv))) (define (slot-index name) + (when (keyword? name) (set! name (keyword->symbol name))) (or (position name slotnames) (error "no such slot in instances of given class" name c))) (let loop ((svsv svsv))
Then one might expect it to be possible for slot names to be keywords in every other use case too (such as slot accessors); but I have custom syntax implemented for those anyway, so far like this:
(-> widget width) ; no colon, because it would look funny; no quote, for convenience
(=> widget width: 50)
I'd rather not have to wrap make in something custom, because it's already such a nice name.
In Chicken 4 it wasn't an issue, because (symbol? foo:) is true.
Attachments (2)
Change History (6)
comment:2 Changed 3 years ago by
So maybe another way to fix this would be to add a parameter or something that I can set in my eggs to make keywords be symbols again, if doing that would not break other code. (or maybe there already is such a thing?)
comment:3 Changed 3 months ago by
Owner: | set to felix winkelmann |
---|---|
Status: | new → accepted |
Changed 3 months ago by
Attachment: | workaround.scm added |
---|
But Felix tells me it breaks existing examples to make this change. So here's my workaround: the class _must_ be defined with keyword slots, and then the -> operator can be adjusted to convert plain symbols to keywords.
Another chicken 4 trick was to define public slots as keywords and private ones as symbols.
(define-class <rect> () ((x: 0) (y: 0) (w: 10) (h: 10) (private 'foo)))
(define r (make <rect> x: 50 'private "whee"))
(slot-ref r x:)
(slot-ref r 'private)
then I could make private slots inaccessible and convert public slot names to keywords in the (-> ...) syntax:
(-> r x) ; no colon needed
(-> r private) ; not allowed
(raspbian still has chicken 4, so I was able to reproduce this on a raspberry pi)