pprint-ns-patch.diff
[CLJ-963] Support pretty printing namespace declarations under code-dispatch Created: 29/Mar/12 Updated: 01/Sep/12 Resolved: 01/Sep/12 |
|
| Status: | Closed |
| Project: | Clojure |
| Component/s: | None |
| Affects Version/s: | Release 1.2, Release 1.3 |
| Fix Version/s: | Release 1.5 |
| Type: | Defect | Priority: | Minor |
| Reporter: | Tom Faulhaber | Assignee: | Stuart Sierra |
| Resolution: | Completed | Votes: | 0 |
| Labels: | patch, | ||
| Environment: |
all |
||
| Attachments: |
pprint-ns-patch.diff
|
| Patch: | Code and Test |
| Approval: | Ok |
| Waiting On: | Rich Hickey |
| Description |
|
Currently when you print code with an (ns ) declaration in it, the ns declaration comes out kind of messy. This patch makes that beautiful. |
| Comments |
| Comment by Stuart Sierra [ 24/Aug/12 8:30 AM ] |
|
Screened. It's not perfect, but an improvement. Before the patch: user=> (with-pprint-dispatch code-dispatch (pprint '(ns com.some-example.my-app (:require clojure.string [clojure.set :as set] [clojure.java.io :refer (file reader)]))))
(ns
com.some-example.my-app
(:require
clojure.string
[clojure.set :as set]
[clojure.java.io :refer (file reader)]))
nil
After the patch: user=> (with-pprint-dispatch code-dispatch (pprint '(ns com.some-example.my-app (:require clojure.string [clojure.set :as set] [clojure.java.io :refer (file reader)]))))
(ns com.some-example.my-app
(:require clojure.string [clojure.set :as set]
[clojure.java.io :refer (file reader)]))
nil
|