diff --git a/ElementX/Sources/Screens/HomeScreen/HomeScreenViewModel.swift b/ElementX/Sources/Screens/HomeScreen/HomeScreenViewModel.swift index cee9db83c1..ea3b631b7a 100644 --- a/ElementX/Sources/Screens/HomeScreen/HomeScreenViewModel.swift +++ b/ElementX/Sources/Screens/HomeScreen/HomeScreenViewModel.swift @@ -343,8 +343,6 @@ class HomeScreenViewModel: HomeScreenViewModelType, HomeScreenViewModelProtocol return } - MXLog.verbose("Updating rooms") - var rooms = [HomeScreenRoom]() for summary in roomSummaryProvider.roomListPublisher.value { @@ -353,8 +351,6 @@ class HomeScreenViewModel: HomeScreenViewModelType, HomeScreenViewModelProtocol } state.rooms = rooms - - MXLog.verbose("Finished updating rooms") } private func updateVisibleRange(_ range: Range) { diff --git a/ElementX/Sources/Screens/MessageForwardingScreen/MessageForwardingScreenViewModel.swift b/ElementX/Sources/Screens/MessageForwardingScreen/MessageForwardingScreenViewModel.swift index 9164474268..72398da48f 100644 --- a/ElementX/Sources/Screens/MessageForwardingScreen/MessageForwardingScreenViewModel.swift +++ b/ElementX/Sources/Screens/MessageForwardingScreen/MessageForwardingScreenViewModel.swift @@ -81,8 +81,6 @@ class MessageForwardingScreenViewModel: MessageForwardingScreenViewModelType, Me // MARK: - Private private func updateRooms() { - MXLog.verbose("Updating rooms") - var rooms = [MessageForwardingRoom]() for summary in roomSummaryProvider.roomListPublisher.value { @@ -95,8 +93,6 @@ class MessageForwardingScreenViewModel: MessageForwardingScreenViewModelType, Me } state.rooms = rooms - - MXLog.verbose("Finished updating rooms") } /// The actual range values don't matter as long as they contain the lower diff --git a/ElementX/Sources/Screens/RoomScreen/View/Timeline/TimelineTableViewController.swift b/ElementX/Sources/Screens/RoomScreen/View/Timeline/TimelineTableViewController.swift index 9ebd316db7..42994abe28 100644 --- a/ElementX/Sources/Screens/RoomScreen/View/Timeline/TimelineTableViewController.swift +++ b/ElementX/Sources/Screens/RoomScreen/View/Timeline/TimelineTableViewController.swift @@ -323,7 +323,6 @@ class TimelineTableViewController: UIViewController { snapshot.appendItems(timelineItemsIDs) let currentSnapshot = dataSource.snapshot() - MXLog.verbose("DIFF: \(snapshot.itemIdentifiers.difference(from: currentSnapshot.itemIdentifiers))") // We only animate when new items come at the end of a live timeline, ignoring transitions through empty. let newestItemIdentifier = snapshot.mainItemIdentifiers.first diff --git a/ElementX/Sources/Services/Room/RoomSummary/RoomSummaryProvider.swift b/ElementX/Sources/Services/Room/RoomSummary/RoomSummaryProvider.swift index ecc5866b65..541d85c0c9 100644 --- a/ElementX/Sources/Services/Room/RoomSummary/RoomSummaryProvider.swift +++ b/ElementX/Sources/Services/Room/RoomSummary/RoomSummaryProvider.swift @@ -264,7 +264,6 @@ class RoomSummaryProvider: RoomSummaryProviderProtocol { switch diff { case .append(let values): - let debugIdentifiers = values.map { $0.id() } for (index, value) in values.enumerated() { let summary = buildRoomSummary(from: value) changes.append(.insert(offset: rooms.count + index, element: summary, associatedWith: nil)) @@ -295,7 +294,6 @@ class RoomSummaryProvider: RoomSummaryProviderProtocol { let summary = rooms[Int(index)] changes.append(.remove(offset: Int(index), element: summary, associatedWith: nil)) case .reset(let values): - let debugIdentifiers = values.map { $0.id() } for (index, summary) in rooms.enumerated() { changes.append(.remove(offset: index, element: summary, associatedWith: nil)) } diff --git a/ElementX/Sources/Services/Timeline/RoomTimelineProvider.swift b/ElementX/Sources/Services/Timeline/RoomTimelineProvider.swift index 0df3782234..03fa54ab6b 100644 --- a/ElementX/Sources/Services/Timeline/RoomTimelineProvider.swift +++ b/ElementX/Sources/Services/Timeline/RoomTimelineProvider.swift @@ -94,8 +94,6 @@ class RoomTimelineProvider: RoomTimelineProviderProtocol { span.exit() } - MXLog.verbose("Received timeline diff") - itemProxies = diffs.reduce(itemProxies) { currentItems, diff in guard let collectionDiff = buildDiff(from: diff, on: currentItems) else { MXLog.error("Failed building CollectionDifference from \(diff)") @@ -110,8 +108,6 @@ class RoomTimelineProvider: RoomTimelineProviderProtocol { return updatedItems } - MXLog.verbose("Finished applying diffs, current items (\(itemProxies.count)) : \(itemProxies.map(\.debugIdentifier))") - if let hasLoadedInitialItemsContinuation { hasLoadedInitialItemsContinuation.resume() self.hasLoadedInitialItemsContinuation = nil @@ -126,7 +122,6 @@ class RoomTimelineProvider: RoomTimelineProviderProtocol { case .append: guard let items = diff.append() else { fatalError() } - MXLog.verbose("Append \(items.map(\.debugIdentifier))") for (index, item) in items.enumerated() { let itemProxy = TimelineItemProxy(item: item) @@ -137,32 +132,25 @@ class RoomTimelineProvider: RoomTimelineProviderProtocol { changes.append(.insert(offset: Int(itemProxies.count) + index, element: itemProxy, associatedWith: nil)) } case .clear: - MXLog.verbose("Clear all items") for (index, itemProxy) in itemProxies.enumerated() { changes.append(.remove(offset: index, element: itemProxy, associatedWith: nil)) } case .insert: guard let update = diff.insert() else { fatalError() } - MXLog.verbose("Insert \(update.item.debugIdentifier) at \(update.index)") let itemProxy = TimelineItemProxy(item: update.item) changes.append(.insert(offset: Int(update.index), element: itemProxy, associatedWith: nil)) case .popBack: guard let itemProxy = itemProxies.last else { fatalError() } - MXLog.verbose("Pop Back \(itemProxy.debugIdentifier)") - changes.append(.remove(offset: itemProxies.count - 1, element: itemProxy, associatedWith: nil)) case .popFront: guard let itemProxy = itemProxies.first else { fatalError() } - MXLog.verbose("Pop Front \(itemProxy.debugIdentifier)") - changes.append(.remove(offset: 0, element: itemProxy, associatedWith: nil)) case .pushBack: guard let item = diff.pushBack() else { fatalError() } - - MXLog.verbose("Push Back \(item.debugIdentifier)") + let itemProxy = TimelineItemProxy(item: item) if itemProxy.isMembershipChange { @@ -173,7 +161,6 @@ class RoomTimelineProvider: RoomTimelineProviderProtocol { case .pushFront: guard let item = diff.pushFront() else { fatalError() } - MXLog.verbose("Push Front: \(item.debugIdentifier)") let itemProxy = TimelineItemProxy(item: item) changes.append(.insert(offset: 0, element: itemProxy, associatedWith: nil)) case .remove: @@ -181,13 +168,10 @@ class RoomTimelineProvider: RoomTimelineProviderProtocol { let itemProxy = itemProxies[Int(index)] - MXLog.verbose("Remove \(itemProxy.debugIdentifier) at: \(index)") - changes.append(.remove(offset: Int(index), element: itemProxy, associatedWith: nil)) case .reset: guard let items = diff.reset() else { fatalError() } - MXLog.verbose("Replace all items with \(items.map(\.debugIdentifier))") for (index, itemProxy) in itemProxies.enumerated() { changes.append(.remove(offset: index, element: itemProxy, associatedWith: nil)) } @@ -198,7 +182,6 @@ class RoomTimelineProvider: RoomTimelineProviderProtocol { case .set: guard let update = diff.set() else { fatalError() } - MXLog.verbose("Set \(update.item.debugIdentifier) at index \(update.index)") let itemProxy = TimelineItemProxy(item: update.item) changes.append(.remove(offset: Int(update.index), element: itemProxy, associatedWith: nil)) changes.append(.insert(offset: Int(update.index), element: itemProxy, associatedWith: nil)) diff --git a/ElementX/Sources/Services/Timeline/TimelineProxy.swift b/ElementX/Sources/Services/Timeline/TimelineProxy.swift index dce0dd1e08..38c106f623 100644 --- a/ElementX/Sources/Services/Timeline/TimelineProxy.swift +++ b/ElementX/Sources/Services/Timeline/TimelineProxy.swift @@ -408,7 +408,7 @@ final class TimelineProxy: TimelineProxyProtocol { } func sendReadReceipt(for eventID: String, type: ReceiptType) async -> Result { - MXLog.verbose("Sending read receipt for eventID: \(eventID)") + MXLog.info("Sending read receipt for eventID: \(eventID)") do { try await timeline.sendReadReceipt(receiptType: type, eventId: eventID)