用pr-str代替str在打印对象时改进错误信息

Question

ClojureScript中的许多错误信息包含无效的参数，如下所示

(throw (js/Error. (str "不支持名称： " x)))

如果{{x}}为null，那么产生的错误信息是{{"不支持名称："}}，这有点难以调试。如果用{{pr-str}}包装了{{x}}，那么错误信息就会变得更容易理解：{{"不支持名称：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通过CI和Canary测试（/）

Answer 9

评论由：mfikes发表

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str-2.patch已添加到Patch Tender（i）

Answer 10

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