通过使用pr-str代替str来打印对象，提高错误信息的可读性

Question

ClojureScript中的许多错误信息都包含如下的无效参数：

(throw (js/Error. (str "Doesn't support name: " x)))

如果 {{x}} 为 nil，那么生成的错误信息将是 {{"Doesn't support name: "}}，这在排错中可能有些令人困惑。如果使用 {{pr-str}} 将 {{x}} 包裹起来，则错误信息将是更易于理解的：{{"Doesn't support name: nil"}}。

如果对此有兴趣，我可以准备一个补丁，将这些错误用 pr-str 进行包装。

Answer 1

评论者：dnolen

继续前进

Answer 2

评论者：mfikes

我也对 CLJS-2089 进行了一些思考。在一般情况下，这似乎是正确的事情。

Answer 3

评论者：mfikes

补丁不再适用。

Answer 4

由 slipset 提出的评论：

现在应该已经应用，希望如此

Answer 5

评论者：mfikes

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str.patch 已不再适用

Answer 6

由 slipset 提出的评论：

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str-2.patch 已更新，应该适用

Answer 7

评论者：mfikes

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str-2.patch LGTM（值得好评）

Answer 8

评论者：mfikes

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str-2.patch 通过了持续集成和Canary测试（/）

Answer 9

评论者：mfikes

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str-2.patch 已添加到补丁申请（i）

Answer 10

参考资料：https://clojure.atlassian.net/browse/CLJS-1908 （由 [email protected] 报告）