refactor: keep access level

jessesquires · Oct 22, 2024 · 011ef02 · 011ef02
1 parent 1625327
commit 011ef02
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/Sources/CollectionViewDriver.swift b/Sources/CollectionViewDriver.swift
@@ -38,14 +38,14 @@ public final class CollectionViewDriver: NSObject {
     /// A flow layout delegate object to receive forwarded events.
     public weak var flowLayoutDelegate: UICollectionViewDelegateFlowLayout?
 
-    let _emptyViewProvider: EmptyViewProvider?
+    private let _emptyViewProvider: EmptyViewProvider?
 
     private var _currentEmptyView: UIView?
 
     // Avoiding a strong reference to prevent a possible retain cycle.
     // This is typically the view controller that owns `self` (the driver).
     // The caller is responsible for retaining this object for the lifetime of the driver.
-    private(set) weak var _cellEventCoordinator: CellEventCoordinator?
+    private weak var _cellEventCoordinator: CellEventCoordinator?
 
     private(set) var _dataSource: DiffableDataSource
 
@@ -581,7 +581,7 @@ extension CollectionViewDriver: UICollectionViewDelegateFlowLayout {
 extension CollectionViewDriver {
     override public var debugDescription: String {
         MainActor.assumeIsolated {
-            driverDebugDescription(self)
+            driverDebugDescription(self, self._emptyViewProvider, self._cellEventCoordinator)
         }
     }
 }
diff --git a/Sources/DebugDescriptions.swift b/Sources/DebugDescriptions.swift
@@ -209,15 +209,19 @@ func driverOptionsDebugDescription(_ options: CollectionViewDriverOptions) -> St
 }
 
 @MainActor
-func driverDebugDescription(_ driver: CollectionViewDriver) -> String {
+func driverDebugDescription(
+    _ driver: CollectionViewDriver,
+    _ emptyViewProvider: EmptyViewProvider?,
+    _ cellEventCoordinator: CellEventCoordinator?
+) -> String {
     var output = ""
     debugDescriptionBuilder(
         elements: [
             (.type(CollectionViewDriver.self), 0),
             (.options(driver.options), 2),
             (.viewModel(driver.viewModel), 2),
-            (.field(label: "emptyViewProvider", value: driver._emptyViewProvider), 2),
-            (.field(label: "cellEventCoordinator", value: driver._cellEventCoordinator), 2),
+            (.field(label: "emptyViewProvider", value: emptyViewProvider), 2),
+            (.field(label: "cellEventCoordinator", value: cellEventCoordinator), 2),
             (.field(label: "scrollViewDelegate", value: driver.scrollViewDelegate), 2),
             (.field(label: "flowLayoutDelegate", value: driver.flowLayoutDelegate), 2),
             (.field(label: "view", value: driver.view), 2),