From a38d97896d35e6c828b0125afe379757a6a88b9d Mon Sep 17 00:00:00 2001
From: joshua <Joshua@qelectrotech.org>
Date: Wed, 30 Mar 2022 20:30:57 +0200
Subject: [PATCH] Minor : avoid unnecessary multiple function call

TerminalStripTreeDockWidget::on_m_tree_view_currentItemChanged
call setCurrentStrip only when current strip changed, and not every time
when user click in another item on the tree view.
---
 sources/TerminalStrip/ui/terminalstriptreedockwidget.cpp | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/sources/TerminalStrip/ui/terminalstriptreedockwidget.cpp b/sources/TerminalStrip/ui/terminalstriptreedockwidget.cpp
index dc5d9cb23..8d427fcae 100644
--- a/sources/TerminalStrip/ui/terminalstriptreedockwidget.cpp
+++ b/sources/TerminalStrip/ui/terminalstriptreedockwidget.cpp
@@ -182,7 +182,10 @@ void TerminalStripTreeDockWidget::on_m_tree_view_currentItemChanged(QTreeWidgetI
 			 && current->parent()->type() == TerminalStripTreeWidget::Strip) {
 		strip_ = m_item_strip_H.value(current->parent());
 	}
-	setCurrentStrip(strip_);
+
+	if (strip_ != m_current_strip) {
+		setCurrentStrip(strip_);
+	}
 }
 
 /**