Skip reloading the sidebar after the tree controller rebuilds when the rebuilt version is the same as the previous version. This saves some cycles, and it’s part of fixing #791.

2019-07-28 15:31:17 -07:00 · 2019-07-28 15:31:17 -07:00 · 2441de960f
commit 2441de960f
parent 44ec6a026d
1 changed files with 3 additions and 2 deletions
--- a/Mac/MainWindow/Sidebar/SidebarViewController.swift
+++ b/Mac/MainWindow/Sidebar/SidebarViewController.swift
@ -358,8 +358,9 @@ private extension SidebarViewController {

 	func rebuildTreeAndReloadDataIfNeeded() {
 		if !animatingChanges && !BatchUpdate.shared.isPerforming {
-			treeController.rebuild()
-			outlineView.reloadData()
+			if treeController.rebuild() {
+				outlineView.reloadData()
+			}
 		}
 	}