Reduce the visibility of some API endpoints

Author: dkhalanskyjb

kotlinx-coroutines-core/common/src/Job.kt

@@ -672,3 +672,9 @@ public object NonDisposableHandle : DisposableHandle, ChildHandle {
      */
     override fun toString(): String = "NonDisposableHandle"
 }
+
+private class DisposeOnCompletion(
+    private val handle: DisposableHandle
+) : JobNode() {
+    override fun invoke(cause: Throwable?) = handle.dispose()
+}

kotlinx-coroutines-core/common/src/JobSupport.kt

@@ -145,7 +145,6 @@ public open class JobSupport constructor(active: Boolean) : Job, ChildJob, Paren
             return
         }
         parent.start() // make sure the parent is started
-        @Suppress("DEPRECATION")
         val handle = parent.attachChild(this)
         parentHandle = handle
         // now check our state _after_ registering (see tryFinalizeSimpleState order of actions)
@@ -1182,7 +1181,7 @@ public open class JobSupport constructor(active: Boolean) : Job, ChildJob, Paren
             return parent.getCancellationException()
         }
 
-        protected override fun nameString(): String =
+        override fun nameString(): String =
             "AwaitContinuation"
     }
 
@@ -1379,7 +1378,7 @@ internal class NodeList : LockFreeLinkedListHead(), Incomplete {
         if (DEBUG) getString("Active") else super.toString()
 }
 
-internal class InactiveNodeList(
+private class InactiveNodeList(
     override val list: NodeList
 ) : Incomplete {
     override val isActive: Boolean get() = false
@@ -1415,12 +1414,6 @@ private class ResumeAwaitOnCompletion<T>(
     }
 }
 
-internal class DisposeOnCompletion(
-    private val handle: DisposableHandle
-) : JobNode() {
-    override fun invoke(cause: Throwable?) = handle.dispose()
-}
-
 // -------- invokeOnCancellation nodes
 
 /**
@@ -1439,7 +1432,7 @@ private class InvokeOnCancelling(
     }
 }
 
-internal class ChildHandleNode(
+private class ChildHandleNode(
     @JvmField val childJob: ChildJob
 ) : JobCancellingNode(), ChildHandle {
     override val parent: Job get() = job

kotlinx-coroutines-core/common/src/internal/Atomic.kt

@@ -19,12 +19,6 @@ public abstract class OpDescriptor {
      */
     abstract fun perform(affected: Any?): Any?
 
-    /**
-     * Returns reference to atomic operation that this descriptor is a part of or `null`
-     * if not a part of any [AtomicOp].
-     */
-    abstract val atomicOp: AtomicOp<*>?
-
     override fun toString(): String = "$classSimpleName@$hexAddress" // debug
 }
 
@@ -46,8 +40,6 @@ internal val NO_DECISION: Any = Symbol("NO_DECISION")
 public abstract class AtomicOp<in T> : OpDescriptor() {
     private val _consensus = atomic<Any?>(NO_DECISION)
 
-    override val atomicOp: AtomicOp<*> get() = this
-
     private fun decide(decision: Any?): Any? {
         assert { decision !== NO_DECISION }
         val current = _consensus.value