核心reducers的微小代码整理：使用必要的walk，取消coll的此操作

Question

首先，core.reducers需要clojure.walk :as walk，但没有使用别名。
其次，函数reducer中coll-reduce的两个arity实现使用'this'，而该文件中的类似实现使用'coll'。据我所知，使用'coll'没有区别（所有测试通过，性能没有变化）并且更符合代码的其他部分。

这两者似乎足够小，可以合并为一个整理案例。

Answer 1

评论者：wagjo

你的补丁是错误的。如果你想用coll替换它，你还需要在f1上调用xf。

`(clojure.core.protocols/coll-reduce this f1 (f1))`

变为

`(clojure.core.protocols/coll-reduce coll (xf f1) (f1))`

Answer 2

评论者：ska2342

令人惊讶的是测试套件没有检测到这一点。

Answer 3

评论者：jafingerhut

增加这样的测试是否简单易懂？

Answer 4

评论者：ska2342

我很愿意调查这件事。不过，这可能需要一些日子，我才能找到时间。

Answer 5

参考： https://clojure.atlassian.net/browse/CLJ-1151 (由 ska2342 报告)