Fixed visibility of atomic properties, they are required to be privat…

…e or internal The new atomicfu JVM compiler plugin (since 1.9.20) will explicitly require atomic properties to be private or internal (or to be members of private or internal classes). See: Kotlin/kotlinx-atomicfu#322
Kotlin · Jul 18, 2023 · 9f4485e · 9f4485e
1 parent 5b64a1f
commit 9f4485e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/kotlinx-coroutines-core/concurrent/test/LimitedParallelismConcurrentTest.kt b/kotlinx-coroutines-core/concurrent/test/LimitedParallelismConcurrentTest.kt
@@ -68,7 +68,7 @@ class LimitedParallelismConcurrentTest : TestBase() {
     @Test
     fun testNotDoingDispatchesWhenNoTasksArePresent() = runTest {
         class NaggingDispatcher: CoroutineDispatcher() {
-            val closed = atomic(false)
+            private val closed = atomic(false)
             override fun dispatch(context: CoroutineContext, block: Runnable) {
                 if (closed.value)
                     fail("Dispatcher was closed, but still dispatched a task")

diff --git a/kotlinx-coroutines-core/concurrent/test/MultithreadedDispatcherStressTest.kt b/kotlinx-coroutines-core/concurrent/test/MultithreadedDispatcherStressTest.kt
@@ -9,7 +9,7 @@ import kotlin.coroutines.*
 import kotlin.test.*
 
 class MultithreadedDispatcherStressTest {
-    val shared = atomic(0)
+    private val shared = atomic(0)
 
     /**
      * Tests that [newFixedThreadPoolContext] will not drop tasks when closed.

diff --git a/kotlinx-coroutines-core/jvm/test/JobHandlersUpgradeStressTest.kt b/kotlinx-coroutines-core/jvm/test/JobHandlersUpgradeStressTest.kt
@@ -29,7 +29,7 @@ class JobHandlersUpgradeStressTest : TestBase() {
     @Volatile
     private var job: Job? = null
 
-    class State {
+    internal class State {
         val state = atomic(0)
     }