[CLJ-783] clojure.inspector/inspect-tree doesn't work on sets --patch in the description by Jason Wolfe Created: 28/Apr/11 Updated: 12/Apr/13 |
|
| Status: | Open |
| Project: | Clojure |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Release 1.6 |
| Type: | Enhancement | Priority: | Minor |
| Reporter: | Armando Blancas | Assignee: | Unassigned |
| Resolution: | Unresolved | Votes: | 1 |
| Labels: | None | ||
| Environment: |
Any |
||
| Attachments: |
|
| Patch: | Code |
| Approval: | Ok |
| Waiting On: | Rich Hickey |
| Description |
|
As reported by Jason Wolfe on March 19, 2009 in the clojure group: clojure.inspector/inspect-tree doesn't work on sets; patch attached I was debugging with inspect-tree and noticed that it errors when it I made a small patch (below) that makes inspect-tree work on Cheers, Index: src/clj/clojure/inspector.clj
(defmulti is-leaf collection-tag) -(defmethod is-leaf :map [m]
(defn tree-model [data] |
| Comments |
| Comment by Andy Fingerhut [ 14/Feb/12 12:54 PM ] |
|
Created a properly formatted patch, attached, for Jason's enhancement. I tested it with (inspect-tree (:members (clojure.reflect/reflect java.lang.Math))) and it worked, whereas it had many errors without Jason's changes. |
| Comment by Andy Fingerhut [ 23/Feb/12 11:58 PM ] |
|
Jason Wolfe has signed a CA. Patch applies cleanly with latest master as of Feb 14, 2012. No errors, warnings, or test failures with the patch applied. No doc strings need updating. |
| Comment by Stuart Sierra [ 09/Nov/12 4:12 PM ] |
|
Screened. |