technomancy / swank-clojure

Swank/slime support for clojure
Eclipse Public License 1.0
412 stars 83 forks source link

clj-stacktrace support #89

Closed dmansen closed 12 years ago

dmansen commented 12 years ago

Added support for clj-stacktrace output

technomancy commented 12 years ago

Excellent; I've been wanting this for a long time. Thanks!

Colors are probably pretty difficult since they're handled on the elisp side, but do you have any idea how difficult it would be to support clj-stacktrace's frame alignment as well?

dmansen commented 12 years ago

I have no clue at all, but let me look into that and get back to you

dmansen commented 12 years ago

Oh, do you mean the textual alignment and spacing? Printing it with indentation like clj-stacktrace does? I started playing around with it and ran into some issues...it might take modifications to clj-stacktrace to get at the parts we would want

technomancy commented 12 years ago

Yeah, it looks like clj-stacktrace doesn't expose what we would need as a function. I have commit rights to clj-stacktrace, so I can release a new version that exposes alignment. Feel free to take a swing at it, or if you don't I'll probably give it a shot some time next week.