[CLJ-1020] clojure.inspector/inspect-table gives up when first element of coll is nil Created: 02/Jul/12 Updated: 03/Sep/13
|Affects Version/s:||Release 1.4|
Ubuntu 12.04, Java 7, Clojure 1.4
|Attachments:||clj-1020-inspect-table-skip-nil-rows-patch1.txt clj-1020-inspect-table-skip-nil-rows-patch2.txt inspector.patch|
clojure.inspector/inspect-table gives up when first element of coll is nil. The patch provided is rather trivial...instead of blindly choosing the first element (which might be nil), it would be more convenient to choose the first element that is NOT nil and use its keys for columns...a similar issue exists with clojure.pprint/print-table where the keys of the first element are used (if not provided explicitly). The same is not true for 'inspect-table' though. As a result, one cannot 'inspect' a collection of maps where the first element is nil. My (trivial) patch looks for the first element which is NOT nil and uses its keys instead. Maps have to have the same length anyway so no problems there...
|Comment by Andy Fingerhut [ 12/Jul/12 1:01 PM ]|
clj-1020-inspect-table-skip-nil-rows-patch1.txt of July 12, 2012 is identical to inspector.patch of July 2, 2012, except it is in the desired git format. Proper attribution is given to author Dimitrios Piliouras in the patch.
|Comment by Dimitrios Piliouras [ 02/Aug/13 9:17 AM ]|
I literally copied the contents of "clj-1020-inspect-table-skip-nil-rows-patch1.txt" and pasted them on "clj-1020-inspect-table-skip-nil-rows-patch2.txt" only changing:
(some #(when-not (nil? %) %) data)
much nicer/cleaner and faster since we skip the internal nil-test.