使用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 pondered了一段时间。在一般情况下，这似乎是正确的事情。

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

Answer 10

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