[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/parseedn f42ff98833 13/32: Merge pull request #1 from ak-c
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/parseedn f42ff98833 13/32: Merge pull request #1 from ak-coram/fix-hash-map-printing |
Date: |
Tue, 28 Dec 2021 14:04:42 -0500 (EST) |
branch: elpa/parseedn
commit f42ff988338484815ccd925c8f83a32c5d52319b
Merge: 342359abd1 0ffab01927
Author: Arne Brasseur <arne.brasseur@gmail.com>
Commit: GitHub <noreply@github.com>
Merge pull request #1 from ak-coram/fix-hash-map-printing
Fix hash map printing
---
parseedn.el | 6 +++---
test/parseedn-test.el | 14 +++++++++++++-
2 files changed, 16 insertions(+), 4 deletions(-)
diff --git a/parseedn.el b/parseedn.el
index 7b09f598c6..e969d23d42 100644
--- a/parseedn.el
+++ b/parseedn.el
@@ -141,16 +141,16 @@ TAG-READERS is an optional association list. For more
information, see
(insert " ")
(parseedn-print-seq next))))
-(defun parseedn-print-kvs (map)
+(defun parseedn-print-kvs (map &optional ks)
"Insert hash table MAP as an EDN map into the current buffer."
- (let ((keys (a-keys map)))
+ (let ((keys (or ks (a-keys map))))
(parseedn-print (car keys))
(insert " ")
(parseedn-print (a-get map (car keys)))
(let ((next (cdr keys)))
(when (not (seq-empty-p next))
(insert ", ")
- (parseedn-print-kvs next)))))
+ (parseedn-print-kvs map next)))))
(defun parseedn-print (datum)
"Insert DATUM as EDN into the current buffer.
diff --git a/test/parseedn-test.el b/test/parseedn-test.el
index d9c3ce79d3..9ddbdb4409 100644
--- a/test/parseedn-test.el
+++ b/test/parseedn-test.el
@@ -37,7 +37,19 @@
(should (equal (parseedn-print-str 100) "100"))
(should (equal (parseedn-print-str 1.2) "1.2"))
(should (equal (parseedn-print-str [1 2 3]) "[1 2 3]"))
- (should (equal (parseedn-print-str t) "true")))
+ (should (equal (parseedn-print-str t) "true"))
+ (should (listp (member (parseedn-print-str
+ (let ((ht (make-hash-table)))
+ (puthash :a 1 ht)
+ (puthash :b 2 ht)
+ (puthash :c 3 ht)
+ ht))
+ '("{:a 1, :b 2, :c 3}"
+ "{:a 1, :c 3, :b 2}"
+ "{:b 2, :a 1, :c 3}"
+ "{:b 2, :c 3, :a 1}"
+ "{:c 3, :a 1, :b 2}"
+ "{:c 3, :b 2, :a 1}")))))
(ert-deftest parseedn-read-test ()
(should (equal (parseedn-read-str "true") t)))
- [nongnu] elpa/parseedn 4efa23a851 15/32: oops! fix test., (continued)
- [nongnu] elpa/parseedn 4efa23a851 15/32: oops! fix test., ELPA Syncer, 2021/12/28
- [nongnu] elpa/parseedn 1a3640d298 29/32: Replace `cl-case` with `cond`, ELPA Syncer, 2021/12/28
- [nongnu] elpa/parseedn b00eb42a1c 28/32: Update the required Emacs version, ELPA Syncer, 2021/12/28
- [nongnu] elpa/parseedn adf57f36f4 11/32: Fix printing of hash-maps with multiple entries, ELPA Syncer, 2021/12/28
- [nongnu] elpa/parseedn 17c4b2f658 05/32: Fix a couple of typos, ELPA Syncer, 2021/12/28
- [nongnu] elpa/parseedn 342359abd1 10/32: Merge pull request #2 from clojure-emacs/fix-build-question-mark, ELPA Syncer, 2021/12/28
- [nongnu] elpa/parseedn 7b9ca20b39 22/32: Merge pull request #7 from clojure-emacs/print-uuid-inst, ELPA Syncer, 2021/12/28
- [nongnu] elpa/parseedn ef0f8772bf 01/32: Initial commit, ELPA Syncer, 2021/12/28
- [nongnu] elpa/parseedn 92bf875962 17/32: Merge pull request #5 from clojure-emacs/support-dotted-pairs, ELPA Syncer, 2021/12/28
- [nongnu] elpa/parseedn e5ba280d1f 32/32: Correctly bump versions, ELPA Syncer, 2021/12/28
- [nongnu] elpa/parseedn f42ff98833 13/32: Merge pull request #1 from ak-coram/fix-hash-map-printing,
ELPA Syncer <=
- [nongnu] elpa/parseedn 8f0582da3f 06/32: Improve the commentary front matter, ELPA Syncer, 2021/12/28
- [nongnu] elpa/parseedn 0ffab01927 12/32: Add test to cover the printing of hash-maps, ELPA Syncer, 2021/12/28
- [nongnu] elpa/parseedn 3b1bea1425 27/32: Drop use of map-merge alist, for Emacs 25/26, ELPA Syncer, 2021/12/28
- [nongnu] elpa/parseedn fe8e30a770 31/32: Update CHANGELOG, ELPA Syncer, 2021/12/28