添加临时项谓词

Question

我遇到了想要检查某些东西是否是临时项以确定是否应该调用assoc(image:或assoc或conj)或conj等情况。

此补丁将transient?作为谓词函数添加。

Answer 1

评论者：alexmiller

补丁需要文档字符串和测试。

Answer 2

评论者：devn

Alex：我猜会是这样的情况！为此道歉。我已经更新了补丁。现在它包括了一个文档字符串，并为 #{}, (link: ), 和 {} 测试了 transient?

谢谢！

Answer 3

评论者：alexmiller

谢谢 - 请不要使用标签“补丁”或“测试”，这些已经在补丁字段中涵概了。

Answer 4

评论者：devn

啊，对不起，Alex。我假设你第一次标记为“补丁”是为了将此票务作为仍然需要经过审查的补丁发出。我的错误。

Answer 5

评论人为：jafingerhut

补丁 0001-Add-transient-predicate.patch，日期为 2014 年 3 月 17 日，最新版本的 Clojure master 上应用顺利，但由于新的函数 transient? 缺乏 :added 属性，导致测试失败。请参考 clojure.core 中其他 Clojure 函数的示例。

在运行测试时还产生了一个新的警告

警告：transient? 已经引用： '#'clojure.core/transient? 在命名空间： clojure.test-clojure.data-structures，将被替换为： '#'clojure.test-clojure.data-structures/transient?

在测试文件 data_structures.clj 中有一个较旧的（但等效的）transient? 定义，当将其添加到 clojure.core 时应该删除。

Answer 6

评论者：devn

@Andy，我没有添加 :added 属性的原因是我不知道这个补丁是否会/何时会被接受，因此，我也不太确定它是否会偷偷进入 1.6.X 或 1.7。目前，我将其放入了 1.7。如果它有希望比这更早添加，请通知我，我将进行调整。

关于警告：已经发现了。我已经提交了一个新补丁，该补丁从 data_structures.clj 中移除了私有版本的 transient?。所有测试都已通过。

编辑以添加：根据此评论的最新补丁是 0002-Add-transient-predicate.patch。

Answer 7

评论人为：jafingerhut

补丁 0002-Add-transient-predicate.patch 日期为 2014 年 3 月 22 日，由于今天早上提交的一些更改，不再干净地应用到最新的 Clojure master 上。我尚未检查此补丁是否更新简单。

Answer 8

评论者：devn

我已经将补丁更新为 0003-Add-transient-predicate.patch。此补丁干净地应用到了最新的 master 版本上。

Answer 9

评论人为：jafingerhut

补丁 0003-Add-transient-predicate.patch 日期为 2013 年 8 月 6 日，在 2014 年 8 月 29 日向 Clojure 提交更改后不再干净地应用于最新的 master。在此之前，它已经干净地应用了。

我还没有检查更新此补丁的可能性和复杂性。

Answer 10

评论者：devn

我已经将补丁更新到0004-Add-transient-predicate.patch。这个补丁应用到master的最新版本上没有任何问题。

Answer 11

参考： https://clojure.atlassian.net/browse/CLJ-1386（由devn报告）