Clojure.pprint/pprint 在使用代码分发表时出现bug

Question

在Clojure和ClojureScript中，使用代码分发表并打印let时，在clojure.pprint/pprint中似乎存在一个bug。

Clojure 1.11.1
user=> (require '[clojure.pprint :as pp])

user=> 
(binding [pp/*print-pprint-dispatch* pp/code-dispatch]
  (pp/pprint
   '(let)))

((let)) ;; prints some extra parenthesis

(binding [pp/*print-pprint-dispatch* pp/code-dispatch]
  (pp/pprint
   '(let*)))

(let*) ;; doesn't happen when pprinting any other form

Answer 1

所以问题在于 clojure.pprint/pprint-let 中的 pprint-logical-block 将两个分支都包裹起来，而它不应该这样做，因为 pprint-simple-code-list 已经生成了括号。

这里添加一个补丁

From 1761194420c8befe62a6e12cabfd4483b645b703 Mon Sep 17 00:00:00 2001                                
From: Juan Monetta <[email protected]>                                                             
Date: Fri, 10 Nov 2023 11:19:02 -0300                                                                 
Subject: [PATCH] Fix clojure.pprint pprint-let when form doesn't contain                               
 bindings                                                                                              
                                                                                                      
---                                                                                                     
 src/clj/clojure/pprint/dispatch.clj | 15 ++++++++-------                                               
 1 file changed, 8 insertions(+), 7 deletions(-)                                                         
                                                                                                      
diff --git a/src/clj/clojure/pprint/dispatch.clj b/src/clj/clojure/pprint/dispatch.clj                
index 965c0b25..cd1c2b29 100644                                                                        
--- a/src/clj/clojure/pprint/dispatch.clj                                                              
+++ b/src/clj/clojure/pprint/dispatch.clj                                                             
@@ -335,13 +335,14 @@                                                                                 
                                                                                                       
 (defn- pprint-let [alis]                                                                              
   (let [base-sym (first alis)]                                                                        
-    (pprint-logical-block :prefix "(" :suffix ")"                                                    
-      (if (and (next alis) (vector? (second alis)))                                                    
-        (do                                                                                            
-          ((formatter-out "~w ~1I~@_") base-sym)                                                     
-          (pprint-binding-form (second alis))                                                        
-          ((formatter-out " ~_~{~w~^ ~_~}") (next (rest alis))))                                      
-        (pprint-simple-code-list alis)))))                                                            
+    (if (and (next alis) (vector? (second alis)))                                                    
+      (pprint-logical-block                                                                          
+       :prefix "(" :suffix ")"                                                                        
+       (do                                                                                            
+         ((formatter-out "~w ~1I~@_") base-sym)                                                       
+         (pprint-binding-form (second alis))                                                           
+         ((formatter-out " ~_~{~w~^ ~_~}") (next (rest alis)))))                                       
+      (pprint-simple-code-list alis))))                                                                
                                                                                                         
                                                                                                         
 ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;                    
--   

Comments

您想向Jira票据添加官方补丁吗？

---

我不确定我能否做到，因为我没有Jira用户。

---

如果您愿意，可以遵循贡献说明[这里](https://clojure.org/dev/dev)。没有压力这样做，我只是想提出建议，因为您已经找到了一个可能的解决方案。

Answer 2

为跟踪此问题已创建了Jira工单https://clojure.atlassian.net/browse/CLJ-2816。