From 07234e53955a214c84f920db8005a193a755428b Mon Sep 17 00:00:00 2001
From: Michael Gregorius <michael.gregorius.git@arcor.de>
Date: Sun, 12 Jan 2025 19:27:37 +0100
Subject: [PATCH] Introduce getOSSpecificModifierKey

Introduce `getOSSpecificModifierKey` to get rid of some more ifdefs.
---
 include/OperatingSystemHelpers.h         | 11 +++++++++++
 src/gui/AutomatableModelView.cpp         |  8 ++------
 src/gui/clips/ClipView.cpp               | 13 ++-----------
 src/gui/tracks/TrackOperationsWidget.cpp |  9 ++-------
 4 files changed, 17 insertions(+), 24 deletions(-)

diff --git a/include/OperatingSystemHelpers.h b/include/OperatingSystemHelpers.h
index 1255e3e559b..d5561d3c2eb 100644
--- a/include/OperatingSystemHelpers.h
+++ b/include/OperatingSystemHelpers.h
@@ -27,6 +27,8 @@
 
 #include "lmmsconfig.h"
 
+#include "qnamespace.h"
+
 
 namespace lmms
 {
@@ -62,6 +64,15 @@ constexpr const char* getOSSppecificModifierKeyString()
 #endif
 }
 
+constexpr Qt::KeyboardModifier getOSSpecificModifierKey()
+{
+#ifdef LMMS_BUILD_APPLE
+	return Qt::AltModifier;
+#else
+	return Qt::ControlModifier;
+#endif    
+}
+
 } // namespace lmms
 
 #endif // LMMS_OPERATINGSYSTEMHELPERS_H
diff --git a/src/gui/AutomatableModelView.cpp b/src/gui/AutomatableModelView.cpp
index ffdef35f5d7..fbbf60ea694 100644
--- a/src/gui/AutomatableModelView.cpp
+++ b/src/gui/AutomatableModelView.cpp
@@ -32,6 +32,7 @@
 #include "embed.h"
 #include "GuiApplication.h"
 #include "MainWindow.h"
+#include "OperatingSystemHelpers.h"
 #include "StringPairDrag.h"
 #include "Clipboard.h"
 
@@ -171,12 +172,7 @@ void AutomatableModelView::unsetModel()
 
 void AutomatableModelView::mousePressEvent( QMouseEvent* event )
 {
-	if( event->button() == Qt::LeftButton &&
-#ifdef LMMS_BUILD_APPLE
-		event->modifiers() & Qt::AltModifier )
-#else
-		event->modifiers() & Qt::ControlModifier )
-#endif
+	if (event->button() == Qt::LeftButton && event->modifiers() & getOSSpecificModifierKey())
 	{
 		new gui::StringPairDrag( "automatable_model", QString::number( modelUntyped()->id() ), QPixmap(), widget() );
 		event->accept();
diff --git a/src/gui/clips/ClipView.cpp b/src/gui/clips/ClipView.cpp
index c128f66f907..443abe56cdf 100644
--- a/src/gui/clips/ClipView.cpp
+++ b/src/gui/clips/ClipView.cpp
@@ -634,12 +634,7 @@ void ClipView::mousePressEvent( QMouseEvent * me )
 		auto pClip = dynamic_cast<PatternClip*>(m_clip);
 		const bool knifeMode = m_trackView->trackContainerView()->knifeMode();
 
-#ifdef LMMS_BUILD_APPLE
-		if ( me->modifiers() & Qt::AltModifier &&
-#else
-		if ( me->modifiers() & Qt::ControlModifier &&
-#endif
-			!(sClip && knifeMode) )
+		if (me->modifiers() & getOSSpecificModifierKey() && !(sClip && knifeMode))
 		{
 			if( isSelected() )
 			{
@@ -830,11 +825,7 @@ void ClipView::mouseMoveEvent( QMouseEvent * me )
 		}
 	}
 
-#ifdef LMMS_BUILD_APPLE
-	if( me->modifiers() & Qt::AltModifier )
-#else
-	if( me->modifiers() & Qt::ControlModifier )
-#endif
+	if (me->modifiers() & getOSSpecificModifierKey())
 	{
 		delete m_hint;
 		m_hint = nullptr;
diff --git a/src/gui/tracks/TrackOperationsWidget.cpp b/src/gui/tracks/TrackOperationsWidget.cpp
index 9ead32a3bff..8220ef7494e 100644
--- a/src/gui/tracks/TrackOperationsWidget.cpp
+++ b/src/gui/tracks/TrackOperationsWidget.cpp
@@ -181,13 +181,8 @@ TrackOperationsWidget::TrackOperationsWidget( TrackView * parent ) :
  */
 void TrackOperationsWidget::mousePressEvent( QMouseEvent * me )
 {
-	if( me->button() == Qt::LeftButton &&
-#ifdef LMMS_BUILD_APPLE
-		me->modifiers() & Qt::AltModifier &&
-#else
-		me->modifiers() & Qt::ControlModifier &&
-#endif
-			m_trackView->getTrack()->type() != Track::Type::Pattern)
+	if (me->button() == Qt::LeftButton && me->modifiers() & getOSSpecificModifierKey() &&
+		m_trackView->getTrack()->type() != Track::Type::Pattern)
 	{
 		DataFile dataFile( DataFile::Type::DragNDropData );
 		m_trackView->getTrack()->saveState( dataFile, dataFile.content() );