核心.reducers中的小量代码清理：使用必需的walk，删除此coll

Question

首先，core.reducers需要clojure.walk:as walk，但并未使用别名。
其次，函数reducer中的coll-reduce有两个参数实现使用'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 报告）