diff options
author | Samuel Gaist <samuel.gaist@idiap.ch> | 2019-02-04 00:09:21 +0100 |
---|---|---|
committer | Samuel Gaist <samuel.gaist@idiap.ch> | 2019-02-05 10:43:28 +0000 |
commit | c59c6cc7b8963e002aec32a67030f7596f8eb7e9 (patch) | |
tree | 10192b875e9b9aa310bdc92236fc73008df56342 | |
parent | a63280c8c77df0f09f48df7d154b66317973e990 (diff) | |
download | qttools-c59c6cc7b8963e002aec32a67030f7596f8eb7e9.tar.gz |
makeqpf: migrate to QRegularExpression
This patch updates makeqpf's code to use QRegularExpression in place of
QRegExp which is to be considered deprecated.
Task-number: QTBUG-72594
Change-Id: I9ee0478e64377460f1ae33f96d0613b44a6b2011
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
-rw-r--r-- | src/makeqpf/mainwindow.cpp | 20 |
1 files changed, 12 insertions, 8 deletions
diff --git a/src/makeqpf/mainwindow.cpp b/src/makeqpf/mainwindow.cpp index ce745e825..3e74a54c1 100644 --- a/src/makeqpf/mainwindow.cpp +++ b/src/makeqpf/mainwindow.cpp @@ -37,6 +37,7 @@ #include <QShortcut> #include <QCompleter> #include <QDirModel> +#include <QRegularExpression> #include <QTextCodec> QT_BEGIN_NAMESPACE @@ -236,8 +237,10 @@ void MainWindow::populateCharacterRanges() if (!f.open(QIODevice::ReadOnly | QIODevice::Text)) return; - QRegExp rangeExpr("([0-9a-f]+)\\.\\.([0-9a-f]+); (.+)"); - rangeExpr.setCaseSensitivity(Qt::CaseInsensitive); + QRegularExpression rangeExpr( + QRegularExpression::anchoredPattern("([0-9a-f]+)\\.\\.([0-9a-f]+); (.+)"), + QRegularExpression::CaseInsensitiveOption + ); QString ellipsis(QChar(0x2026)); if (!characterRangeView->fontMetrics().inFont(ellipsis.at(0))) @@ -256,30 +259,31 @@ void MainWindow::populateCharacterRanges() if (line.isEmpty() || line.startsWith(QLatin1Char('#'))) continue; - if (!rangeExpr.exactMatch(line) || rangeExpr.captureCount() != 3) + QRegularExpressionMatch match = rangeExpr.match(line); + if (!match.hasMatch()) continue; QPF::CharacterRange range; bool ok = false; - range.start = rangeExpr.cap(1).toUInt(&ok, /*base*/16); + range.start = match.captured(1).toUInt(&ok, /*base*/16); if (!ok) continue; - range.end = rangeExpr.cap(2).toUInt(&ok, /*base*/16); + range.end = match.captured(2).toUInt(&ok, /*base*/16); if (!ok) continue; if (range.start >= 0xffff || range.end >= 0xffff) continue; - QString description = rangeExpr.cap(3); + QString description = match.captured(3); QListWidgetItem *item = new QListWidgetItem(characterRangeView); QString text = description; text.append(QLatin1String(" (")); - text.append(rangeExpr.cap(1)); + text.append(match.captured(1)); text.append(ellipsis); - text.append(rangeExpr.cap(2)); + text.append(match.captured(2)); text.append(QLatin1String(")")); item->setText(text); item->setCheckState(Qt::Checked); |