Skip to content

Commit 036a1d6

Browse files
refactor: drop SkipEmptyParts conditional compile
since Qt 5.15 needed
1 parent 38e89e6 commit 036a1d6

File tree

6 files changed

+7
-41
lines changed

6 files changed

+7
-41
lines changed

makespec/BUILDVERSION

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
124
1+
125

src/base/LemonUtils.hpp

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,15 +21,9 @@
2121
#define READ_JSON_STR(___x) \
2222
if (json.contains("___x") && json["___x"].isString()) \
2323
___x = json["___x"].toString();
24-
#if QT_VERSION >= QT_VERSION_CHECK(5, 14, 0)
2524
#define READ_JSON_STRLIST(___x) \
2625
if (json.contains("___x") && json["___x"].isString()) \
2726
___x = json["___x"].toString().split(QLatin1Char(';'), Qt::SkipEmptyParts);
28-
#else
29-
#define READ_JSON_STRLIST(___x) \
30-
if (json.contains("___x") && json["___x"].isString()) \
31-
___x = json["___x"].toString().split(QLatin1Char(';'), QString::SkipEmptyParts);
32-
#endif
3327

3428
#define WRITE_JSON(___x) json["___x"] = ___x;
3529
#define WRITE_JSON_STRLIST(___x) json["___x"] = ___x.join(QLatin1Char(';'));

src/base/compiler.cpp

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,6 @@
1111
//
1212
#include "base/LemonUtils.hpp"
1313

14-
#if QT_VERSION >= QT_VERSION_CHECK(5, 14, 0)
15-
#define QT_SkipEmptyParts Qt::SkipEmptyParts
16-
#else
17-
#define QT_SkipEmptyParts QString::SkipEmptyParts
18-
#endif
19-
2014
Compiler::Compiler(QObject *parent) : QObject(parent) {
2115
compilerType = Typical;
2216
timeLimitRatio = 1;
@@ -55,15 +49,15 @@ void Compiler::setCompilerType(Compiler::CompilerType type) { compilerType = typ
5549
void Compiler::setCompilerName(const QString &name) { compilerName = name; }
5650

5751
void Compiler::setSourceExtensions(const QString &extensions) {
58-
sourceExtensions = extensions.split(";", QT_SkipEmptyParts);
52+
sourceExtensions = extensions.split(";", Qt::SkipEmptyParts);
5953
}
6054

6155
void Compiler::setCompilerLocation(const QString &location) { compilerLocation = location; }
6256

6357
void Compiler::setInterpreterLocation(const QString &location) { interpreterLocation = location; }
6458

6559
void Compiler::setBytecodeExtensions(const QString &extensions) {
66-
bytecodeExtensions = extensions.split(";", QT_SkipEmptyParts);
60+
bytecodeExtensions = extensions.split(";", Qt::SkipEmptyParts);
6761
}
6862

6963
void Compiler::setEnvironment(const QProcessEnvironment &env) { environment = env; }
@@ -139,11 +133,7 @@ void Compiler::read(const QJsonObject &json) {
139133

140134
QStringList _environment;
141135
if (json.contains("environment") && json["environment"].isString()) {
142-
#if QT_VERSION >= QT_VERSION_CHECK(5, 14, 0)
143136
_environment = json["environment"].toString().split(QLatin1Char(';'), Qt::SkipEmptyParts);
144-
#else
145-
_environment = json["environment"].toString().split(QLatin1Char(';'), QString::SkipEmptyParts);
146-
#endif
147137
}
148138

149139
for (auto &i : _environment) {

src/base/settings.cpp

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -16,12 +16,6 @@
1616
//
1717
#define LEMON_MODULE_NAME "Setting"
1818

19-
#if QT_VERSION >= QT_VERSION_CHECK(5, 14, 0)
20-
#define QT_SkipEmptyParts Qt::SkipEmptyParts
21-
#else
22-
#define QT_SkipEmptyParts QString::SkipEmptyParts
23-
#endif
24-
2519
Settings::Settings(QObject *parent) : QObject(parent) {}
2620

2721
auto Settings::getDefaultFullScore() const -> int { return defaultFullScore; }
@@ -262,12 +256,12 @@ void Settings::setDefaultOutputFileExtension(const QString &extension) {
262256
}
263257

264258
void Settings::setInputFileExtensions(const QString &extensions) {
265-
inputFileExtensions = extensions.split(";", QT_SkipEmptyParts);
259+
inputFileExtensions = extensions.split(";", Qt::SkipEmptyParts);
266260
DEBUG("Set InputFile Extensions to " + extensions);
267261
}
268262

269263
void Settings::setOutputFileExtensions(const QString &extensions) {
270-
outputFileExtensions = extensions.split(";", QT_SkipEmptyParts);
264+
outputFileExtensions = extensions.split(";", Qt::SkipEmptyParts);
271265
DEBUG("Set OutputFile Extensions to " + extensions);
272266
}
273267

src/core/assignmentthread.cpp

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -19,12 +19,6 @@
1919
#include "core/testcase.h"
2020
#include <utility>
2121

22-
#if QT_VERSION >= QT_VERSION_CHECK(5, 14, 0)
23-
#define QT_SkipEmptyParts Qt::SkipEmptyParts
24-
#else
25-
#define QT_SkipEmptyParts QString::SkipEmptyParts
26-
#endif
27-
2822
AssignmentThread::AssignmentThread(QObject *parent) : QThread(parent) {
2923
moveToThread(this);
3024
curTestCaseIndex = 0;
@@ -220,7 +214,7 @@ auto AssignmentThread::traditionalTaskPrepare() -> bool {
220214
// TODO: 需要重构代码来处理含空格路径问题
221215

222216
compiler->start(i->getCompilerLocation(),
223-
arguments.split(QLatin1Char(' '), QT_SkipEmptyParts));
217+
arguments.split(QLatin1Char(' '), Qt::SkipEmptyParts));
224218

225219
if (! compiler->waitForStarted(-1)) {
226220
compileState = InvalidCompiler;

src/detaildialog.cpp

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -22,12 +22,6 @@
2222
#include <QMessageBox>
2323
#include <QScrollBar>
2424

25-
#if QT_VERSION >= QT_VERSION_CHECK(5, 14, 0)
26-
#define QT_SkipEmptyParts Qt::SkipEmptyParts
27-
#else
28-
#define QT_SkipEmptyParts QString::SkipEmptyParts
29-
#endif
30-
3125
DetailDialog::DetailDialog(QWidget *parent) : QDialog(parent), ui(new Ui::DetailDialog) {
3226
ui->setupUi(this);
3327
connect(ui->detailViewer, &QTextBrowser::anchorClicked, this, &DetailDialog::anchorClicked);
@@ -259,7 +253,7 @@ void DetailDialog::showDialog() {
259253
}
260254

261255
void DetailDialog::anchorClicked(const QUrl &url) {
262-
QStringList list = url.path().split(' ', QT_SkipEmptyParts);
256+
QStringList list = url.path().split(' ', Qt::SkipEmptyParts);
263257

264258
if (list[0] == "Rejudge") {
265259
auto *dialog = new JudgingDialog(this);

0 commit comments

Comments
 (0)