Fix concurrent remove

ddae4c10 · Him188 · b38f8252 · ddae4c10
Commit ddae4c10 authored Feb 02, 2020 by Him188
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

mirai-core/src/commonMain/kotlin/net.mamoe.mirai/utils/LockFreeLinkedList.kt ...onMain/kotlin/net.mamoe.mirai/utils/LockFreeLinkedList.kt +4 -1

No files found.
--- a/mirai-core/src/commonMain/kotlin/net.mamoe.mirai/utils/LockFreeLinkedList.kt
+++ b/mirai-core/src/commonMain/kotlin/net.mamoe.mirai/utils/LockFreeLinkedList.kt
@@ -132,7 +132,10 @@ open class LockFreeLinkedList<E> {
            if (toRemove.isRemoved()) {
                continue
            }
-            toRemove.removed.value = true // logically remove: all the operations will recognize this node invalid
+            if (!toRemove.removed.compareAndSet(false, true)) {
+                // logically remove: all the operations will recognize this node invalid
+                continue
+            }
            // physically remove: try to fix the link