Details
-
Type:
Defect
-
Status:
Closed
-
Priority:
Minor
-
Resolution: Completed
-
Affects Version/s: Release 1.3
-
Fix Version/s: Release 1.4
-
Component/s: None
-
Labels:None
-
Patch:Code and Test
-
Approval:Ok
Description
If the keywords of a protocol's method map are namespaced, the map is accepted, but lookup fails since lookup uses non-namespaced keywords.
See TLOG-4 for an actual case of this being an issue.
Work-around for namespaced keywords with extend:
don't use namespaced keywords
Work-around for syntax-quoting with extend-type or extend-protocol:
use extend with non-namespaced keywords
Possible solutions:
- Inside extend, remove namespace of keywords
- or
- Inside extend, error on namespaced keywords.
- Inside emit-hinted-impl, only grab name portion of symbols before converting to keyword.
Attachments
Activity
Fogus
made changes -
| Field | Original Value | New Value |
|---|---|---|
| Fix Version/s | Release 1.4 [ 10040 ] |
Alexander Taggart
made changes -
| Assignee | Alexander Taggart [ ataggart ] |
Alexander Taggart
made changes -
| Status | Open [ 1 ] | In Progress [ 3 ] |
Alexander Taggart
made changes -
| Attachment | clj-845-1.patch [ 10377 ] | |
| Patch | Code and Test |
Alexander Taggart
made changes -
| Approval | Test |
Alexander Taggart
made changes -
| Status | In Progress [ 3 ] | Open [ 1 ] |
Stuart Sierra
made changes -
| Approval | Test [ 10013 ] | Screened [ 10004 ] |
Rich Hickey
made changes -
| Approval | Screened [ 10004 ] | Ok [ 10007 ] |
Stuart Sierra
made changes -
| Resolution | Completed [ 1 ] | |
| Status | Open [ 1 ] | Resolved [ 5 ] |
Stuart Halloway
made changes -
| Status | Resolved [ 5 ] | Closed [ 6 ] |