Fix put and putAll preserving map instance in case of no-operation.

ilya-g · ilya-g · commit 7c8a8dcdc6df · 2017-02-20T02:46:32.000+03:00
diff --git a/kotlinx-collections-immutable/tests/src/test/kotlin/kotlinx.collections.immutable/ImmutableMapTest.kt b/kotlinx-collections-immutable/tests/src/test/kotlin/kotlinx.collections.immutable/ImmutableMapTest.kt
@@ -139,8 +139,8 @@ abstract class ImmutableMapTest {
         with(map) {
             testNoOperation({ remove("y") }, { remove("y") })
             testNoOperation({ remove("x", 2) }, { remove("x", 2) })
-//            testNoOperation({ put("x", 1) }, { put("x", 1) })     // does not hold
-//            testNoOperation({ putAll(this) }, { putAll(this) })   // does not hold
+            testNoOperation({ put("x", 1) }, { put("x", 1) })     // does not hold
+            testNoOperation({ putAll(this) }, { putAll(this) })   // does not hold
             testNoOperation({ putAll(emptyMap()) }, { putAll(emptyMap()) })
         }
     }
diff --git a/pcollections b/pcollections
@@ -1 +1 @@
-Subproject commit 9d255a73c6efc95673a8e93974941a15d8f36950
+Subproject commit 6c5e040cdff516b843cbadff2ec3e04df1aa8a7c

Original file line number	Diff line number	Diff line change
`@@ -139,8 +139,8 @@ abstract class ImmutableMapTest {`
`139`	`139`	`with(map) {`
`140`	`140`	`testNoOperation({ remove("y") }, { remove("y") })`
`141`	`141`	`testNoOperation({ remove("x", 2) }, { remove("x", 2) })`
`142`		`-// testNoOperation({ put("x", 1) }, { put("x", 1) }) // does not hold`
`143`		`-// testNoOperation({ putAll(this) }, { putAll(this) }) // does not hold`
	`142`	`+ testNoOperation({ put("x", 1) }, { put("x", 1) }) // does not hold`
	`143`	`+ testNoOperation({ putAll(this) }, { putAll(this) }) // does not hold`
`144`	`144`	`testNoOperation({ putAll(emptyMap()) }, { putAll(emptyMap()) })`
`145`	`145`	`}`
`146`	`146`	`}`