[utilities/krusader] app: Dissolve no longer needed compat helper macro SKIP_EMPTY_PARTS

Toni Asensi Esteve <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit 19886e088d14536229409e1e33e6e8df3dc7c3a9 by Toni Asensi Esteve, on behalf of Friedrich W. H. Kossebau.
Committed on 19/07/2026 at 23:02.
Pushed by asensi into branch 'master'.

Dissolve no longer needed compat helper macro SKIP_EMPTY_PARTS

M  +1    -2    app/GUI/krremoteencodingmenu.cpp
M  +1    -2    app/Konfigurator/kgdependencies.cpp
M  +0    -14   app/compat.h

https://invent.kde.org/utilities/krusader/-/commit/19886e088d14536229409e1e33e6e8df3dc7c3a9

diff --git a/app/GUI/krremoteencodingmenu.cpp b/app/GUI/krremoteencodingmenu.cpp
index 0a4a8841e..258641b6e 100644
--- a/app/GUI/krremoteencodingmenu.cpp
+++ b/app/GUI/krremoteencodingmenu.cpp
@@ -23,7 +23,6 @@
 
 #include "../Panel/krpanel.h"
 #include "../Panel/panelfunc.h"
-#include "../compat.h"
 #include "../icon.h"
 #include "../krglobal.h"
 
@@ -170,7 +169,7 @@ void KrRemoteEncodingMenu::chooseDefault()
     // settings here since it affects what will be matched.
     KConfig config(("kio_" + currentURL.scheme() + "rc").toLatin1());
 
-    QStringList partList = currentURL.host().split('.', SKIP_EMPTY_PARTS);
+    QStringList partList = currentURL.host().split('.', Qt::SkipEmptyParts);
     if (!partList.isEmpty()) {
         partList.erase(partList.begin());
 
diff --git a/app/Konfigurator/kgdependencies.cpp b/app/Konfigurator/kgdependencies.cpp
index 954e2d884..b55a21e1e 100644
--- a/app/Konfigurator/kgdependencies.cpp
+++ b/app/Konfigurator/kgdependencies.cpp
@@ -6,7 +6,6 @@
 */
 
 #include "kgdependencies.h"
-#include "../compat.h"
 #include "../krglobal.h"
 #include "../krservices.h"
 
@@ -114,7 +113,7 @@ void KgDependencies::addApplication(const QString &name, QGridLayout *grid, int
     QString defaultValue = KrServices::fullPathName(name);
 
     if (defaultValue.isEmpty()) {
-        QStringList list = additionalList.split(',', SKIP_EMPTY_PARTS);
+        QStringList list = additionalList.split(',', Qt::SkipEmptyParts);
         for (int i = 0; i != list.count(); i++)
             if (!KrServices::fullPathName(list[i]).isEmpty()) {
                 defaultValue = KrServices::fullPathName(list[i]);
diff --git a/app/compat.h b/app/compat.h
index da01d07b4..c6889714e 100644
--- a/app/compat.h
+++ b/app/compat.h
@@ -7,20 +7,6 @@
 #ifndef _COMPAT_H_
 #define _COMPAT_H_
 
-/**
- * QString::split(QChar sep, QString::SplitBehavior behavior, Qt::CaseSensitivity cs = Qt::CaseSensitive)
- * was made obsoleted in QT 5.15 in favor of the namespaced Qt::endl
- *
- * https://doc.qt.io/qt-5.15/qstring-obsolete.html#split-2
- *
- * This can be removed when the qt minimum version required will be >= 5.15
- */
-#if QT_VERSION >= QT_VERSION_CHECK(5, 15, 0)
-#define SKIP_EMPTY_PARTS Qt::SkipEmptyParts
-#else
-#define SKIP_EMPTY_PARTS QString::SkipEmptyParts
-#endif
-
 /**
  * QTextSteam::endl() was made obsoleted in QT 5.15 in
  * favor of the namespaced Qt::endl
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.