使用pr-str代替str打印对象以提高错误消息

Question

ClojureScript中的许多错误消息包含如此类的不合法参数

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

如果{{x}}为nil，则生成的错误消息是{{"Doesn't support name: "}}，这会对调试造成一定的困惑。如果{{x}}被{{pr-str}}包裹，则错误消息会是更容易理解的：{{"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 通过 CI 和 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](https://clojure.atlassian.net/browse/CLJS-1908)（由 [email protected] 报告）