Skip to content

GitLab

Kate
Kate
Commit d29db43e authored by Jan Paul Batrina's avatar Jan Paul Batrina Committed by Kåre Särs
Browse files
Options

[gdbplugin] Port QRegExp to QRegular Expression

parent 1a5ab0fb
Hide whitespace changes
Inline Side-by-side
Showing with 70 additions and 62 deletions
addons/gdbplugin/debugview.cpp
View file @ d29db43e
......@@ -24,7 +24,7 @@
#include "debugview.h"
#include <QFile>
#include <QRegExp>
#include <QRegularExpression>
#include <QTimer>
#include <klocalizedstring.h>
......@@ -262,23 +262,24 @@ void DebugView::slotContinue()
issueCommand(QStringLiteral("continue"));
}
static QRegExp breakpointList(QStringLiteral("Num\\s+Type\\s+Disp\\s+Enb\\s+Address\\s+What.*"));
static QRegExp breakpointListed(QStringLiteral("(\\d)\\s+breakpoint\\s+keep\\sy\\s+0x[\\da-f]+\\sin\\s.+\\sat\\s([^:]+):(\\d+).*"));
static QRegExp stackFrameAny(QStringLiteral("#(\\d+)\\s(.*)"));
static QRegExp stackFrameFile(QStringLiteral("#(\\d+)\\s+(?:0x[\\da-f]+\\s*in\\s)*(\\S+)(\\s\\(.*\\)) at ([^:]+):(\\d+).*"));
static QRegExp changeFile(QStringLiteral("(?:(?:Temporary\\sbreakpoint|Breakpoint)\\s*\\d+,\\s*|0x[\\da-f]+\\s*in\\s*)?[^\\s]+\\s*\\([^)]*\\)\\s*at\\s*([^:]+):(\\d+).*"));
static QRegExp changeLine(QStringLiteral("(\\d+)\\s+.*"));
static QRegExp breakPointReg(QStringLiteral("Breakpoint\\s+(\\d+)\\s+at\\s+0x[\\da-f]+:\\s+file\\s+([^\\,]+)\\,\\s+line\\s+(\\d+).*"));
static QRegExp breakPointMultiReg(QStringLiteral("Breakpoint\\s+(\\d+)\\s+at\\s+0x[\\da-f]+:\\s+([^\\,]+):(\\d+).*"));
static QRegExp breakPointDel(QStringLiteral("Deleted\\s+breakpoint.*"));
static QRegExp exitProgram(QStringLiteral("(?:Program|.*Inferior.*)\\s+exited.*"));
static QRegExp threadLine(QStringLiteral("\\**\\s+(\\d+)\\s+Thread.*"));
static const QRegularExpression breakpointList(QStringLiteral("\\ANum\\s+Type\\s+Disp\\s+Enb\\s+Address\\s+What.*\\z"));
static const QRegularExpression breakpointListed(QStringLiteral("\\A(\\d)\\s+breakpoint\\s+keep\\sy\\s+0x[\\da-f]+\\sin\\s.+\\sat\\s([^:]+):(\\d+).*\\z"));
static const QRegularExpression stackFrameAny(QStringLiteral("\\A#(\\d+)\\s(.*)\\z"));
static const QRegularExpression stackFrameFile(QStringLiteral("\\A#(\\d+)\\s+(?:0x[\\da-f]+\\s*in\\s)*(\\S+)(\\s\\(.*\\)) at ([^:]+):(\\d+).*\\z"));
static const QRegularExpression changeFile(QStringLiteral("\\A(?:(?:Temporary\\sbreakpoint|Breakpoint)\\s*\\d+,\\s*|0x[\\da-f]+\\s*in\\s*)?[^\\s]+\\s*\\([^)]*\\)\\s*at\\s*([^:]+):(\\d+).*\\z"));
static const QRegularExpression changeLine(QStringLiteral("\\A(\\d+)\\s+.*\\z"));
static const QRegularExpression breakPointReg(QStringLiteral("\\ABreakpoint\\s+(\\d+)\\s+at\\s+0x[\\da-f]+:\\s+file\\s+([^\\,]+)\\,\\s+line\\s+(\\d+).*\\z"));
static const QRegularExpression breakPointMultiReg(QStringLiteral("\\ABreakpoint\\s+(\\d+)\\s+at\\s+0x[\\da-f]+:\\s+([^\\,]+):(\\d+).*\\z"));
static const QRegularExpression breakPointDel(QStringLiteral("\\ADeleted\\s+breakpoint.*\\z"));
static const QRegularExpression exitProgram(QStringLiteral("\\A(?:Program|.*Inferior.*)\\s+exited.*\\z"));
static const QRegularExpression threadLine(QStringLiteral("\\A\\**\\s+(\\d+)\\s+Thread.*\\z"));
void DebugView::processLine(QString line)
{
if (line.isEmpty())
return;
static QRegularExpressionMatch match;
switch (m_state) {
case none:
case ready:
......@@ -289,36 +290,36 @@ void DebugView::processLine(QString line)
break;
case executingCmd:
if (breakpointList.exactMatch(line)) {
if (breakpointList.match(line).hasMatch()) {
m_state = listingBreakpoints;
emit clearBreakpointMarks();
m_breakPointList.clear();
} else if (line.contains(QLatin1String("No breakpoints or watchpoints."))) {
emit clearBreakpointMarks();
m_breakPointList.clear();
} else if (stackFrameAny.exactMatch(line)) {
} else if ((match = stackFrameAny.match(line)).hasMatch()) {
if (m_lastCommand.contains(QLatin1String("info stack"))) {
emit stackFrameInfo(stackFrameAny.cap(1), stackFrameAny.cap(2));
emit stackFrameInfo(match.captured(1), match.captured(2));
} else {
m_subState = (m_subState == normal) ? stackFrameSeen : stackTraceSeen;
m_newFrameLevel = stackFrameAny.cap(1).toInt();
m_newFrameLevel = match.captured(1).toInt();
if (stackFrameFile.exactMatch(line)) {
m_newFrameFile = stackFrameFile.cap(4);
if ((match = stackFrameFile.match(line)).hasMatch()) {
m_newFrameFile = match.captured(4);
}
}
} else if (changeFile.exactMatch(line)) {
m_currentFile = changeFile.cap(1).trimmed();
int lineNum = changeFile.cap(2).toInt();
} else if ((match = changeFile.match(line)).hasMatch()) {
m_currentFile = match.captured(1).trimmed();
int lineNum = match.captured(2).toInt();
if (!m_nextCommands.contains(QLatin1String("continue"))) {
// GDB uses 1 based line numbers, kate uses 0 based...
emit debugLocationChanged(resolveFileName(m_currentFile), lineNum - 1);
}
m_debugLocationChanged = true;
} else if (changeLine.exactMatch(line)) {
int lineNum = changeLine.cap(1).toInt();
} else if ((match = changeLine.match(line)).hasMatch()) {
int lineNum = match.captured(1).toInt();
if (m_subState == stackFrameSeen) {
m_currentFile = m_newFrameFile;
......@@ -328,21 +329,21 @@ void DebugView::processLine(QString line)
emit debugLocationChanged(resolveFileName(m_currentFile), lineNum - 1);
}
m_debugLocationChanged = true;
} else if (breakPointReg.exactMatch(line)) {
} else if ((match = breakPointReg.match(line)).hasMatch()) {
BreakPoint breakPoint;
breakPoint.number = breakPointReg.cap(1).toInt();
breakPoint.file = resolveFileName(breakPointReg.cap(2));
breakPoint.line = breakPointReg.cap(3).toInt();
breakPoint.number = match.captured(1).toInt();
breakPoint.file = resolveFileName(match.captured(2));
breakPoint.line = match.captured(3).toInt();
m_breakPointList << breakPoint;
emit breakPointSet(breakPoint.file, breakPoint.line - 1);
} else if (breakPointMultiReg.exactMatch(line)) {
} else if ((match = breakPointMultiReg.match(line)).hasMatch()) {
BreakPoint breakPoint;
breakPoint.number = breakPointMultiReg.cap(1).toInt();
breakPoint.file = resolveFileName(breakPointMultiReg.cap(2));
breakPoint.line = breakPointMultiReg.cap(3).toInt();
breakPoint.number = match.captured(1).toInt();
breakPoint.file = resolveFileName(match.captured(2));
breakPoint.line = match.captured(3).toInt();
m_breakPointList << breakPoint;
emit breakPointSet(breakPoint.file, breakPoint.line - 1);
} else if (breakPointDel.exactMatch(line)) {
} else if (breakPointDel.match(line).hasMatch()) {
line.remove(QStringLiteral("Deleted breakpoint"));
line.remove(QLatin1Char('s')); // in case of multiple breakpoints
QStringList numbers = line.split(QLatin1Char(' '), QString::SkipEmptyParts);
......@@ -355,7 +356,7 @@ void DebugView::processLine(QString line)
}
}
}
} else if (exitProgram.exactMatch(line) || line.contains(QLatin1String("The program no longer exists")) || line.contains(QLatin1String("Kill the program being debugged"))) {
} else if (exitProgram.match(line).hasMatch() || line.contains(QLatin1String("The program no longer exists")) || line.contains(QLatin1String("Kill the program being debugged"))) {
// if there are still commands to execute remove them to remove unneeded output
// except if the "kill was for "re-run"
if ((!m_nextCommands.empty()) && !m_nextCommands[0].contains(QLatin1String("file"))) {
......@@ -374,12 +375,12 @@ void DebugView::processLine(QString line)
}
break;
case listingBreakpoints:
if (breakpointListed.exactMatch(line)) {
case listingBreakpoints:;
if ((match = breakpointListed.match(line)).hasMatch()) {
BreakPoint breakPoint;
breakPoint.number = breakpointListed.cap(1).toInt();
breakPoint.file = resolveFileName(breakpointListed.cap(2));
breakPoint.line = breakpointListed.cap(3).toInt();
breakPoint.number = match.captured(1).toInt();
breakPoint.file = resolveFileName(match.captured(2));
breakPoint.line = match.captured(3).toInt();
m_breakPointList << breakPoint;
emit breakPointSet(breakPoint.file, breakPoint.line - 1);
} else if (PromptStr == line) {
......@@ -417,16 +418,16 @@ void DebugView::processLine(QString line)
m_state = ready;
emit stackFrameInfo(QString(), QString());
QTimer::singleShot(0, this, &DebugView::issueNextCommand);
} else if (stackFrameAny.exactMatch(line)) {
emit stackFrameInfo(stackFrameAny.cap(1), stackFrameAny.cap(2));
} else if ((match = stackFrameAny.match(line)).hasMatch()) {
emit stackFrameInfo(match.captured(1), match.captured(2));
}
break;
case infoThreads:
if (PromptStr == line) {
m_state = ready;
QTimer::singleShot(0, this, &DebugView::issueNextCommand);
} else if (threadLine.exactMatch(line)) {
emit threadInfo(threadLine.cap(1).toInt(), (line[0] == QLatin1Char('*')));
} else if ((match = threadLine.match(line)).hasMatch()) {
emit threadInfo(match.captured(1).toInt(), (line[0] == QLatin1Char('*')));
}
break;
}
......
addons/gdbplugin/localsview.cpp
View file @ d29db43e
......@@ -70,12 +70,12 @@ void LocalsView::createWrappedItem(QTreeWidget *parent, const QString &name, con
void LocalsView::addLocal(const QString &vString)
{
static QRegExp isValue(QStringLiteral("(\\S*)\\s=\\s(.*)"));
static QRegExp isStruct(QStringLiteral("\\{\\S*\\s=\\s.*"));
static QRegExp isStartPartial(QStringLiteral("\\S*\\s=\\s\\S*\\s=\\s\\{"));
static QRegExp isPrettyQList(QStringLiteral("\\s*\\[\\S*\\]\\s=\\s\\S*"));
static QRegExp isPrettyValue(QStringLiteral("(\\S*)\\s=\\s(\\S*)\\s=\\s(.*)"));
static QRegExp isThisValue(QStringLiteral("\\$\\d+"));
static const QRegularExpression isValue(QStringLiteral("\\A(\\S*)\\s=\\s(.*)\\z"));
static const QRegularExpression isStruct(QStringLiteral("\\A\\{\\S*\\s=\\s.*\\z"));
static const QRegularExpression isStartPartial(QStringLiteral("\\A\\S*\\s=\\s\\S*\\s=\\s\\{\\z"));
static const QRegularExpression isPrettyQList(QStringLiteral("\\A\\s*\\[\\S*\\]\\s=\\s\\S*\\z"));
static const QRegularExpression isPrettyValue(QStringLiteral("\\A(\\S*)\\s=\\s(\\S*)\\s=\\s(.*)\\z"));
static const QRegularExpression isThisValue(QStringLiteral("\\A\\$\\d+\\z"));
if (m_allAdded) {
clear();
......@@ -86,11 +86,14 @@ void LocalsView::addLocal(const QString &vString)
m_allAdded = true;
return;
}
if (isStartPartial.exactMatch(vString)) {
QRegularExpressionMatch match = isStartPartial.match(vString);
if (match.hasMatch()) {
m_local = vString;
return;
}
if (isPrettyQList.exactMatch(vString)) {
match = isPrettyQList.match(vString);
if (match.hasMatch()) {
m_local += vString.trimmed();
if (m_local.endsWith(QLatin1Char(',')))
m_local += QLatin1Char(' ');
......@@ -107,24 +110,27 @@ void LocalsView::addLocal(const QString &vString)
if (vString == QLatin1String("No symbol table info available.")) {
return; /* this is not an error */
}
if (!isValue.exactMatch(vString)) {
match = isValue.match(vString);
if (!match.hasMatch()) {
qDebug() << "Could not parse:" << vString;
return;
}
symbolAndValue << isValue.cap(1);
symbolAndValue << match.captured(1);
value = match.captured(2);
// check out for "print *this"
if (isThisValue.exactMatch(symbolAndValue[0])) {
match = isThisValue.match(symbolAndValue[0]);
if (match.hasMatch()) {
symbolAndValue[0] = QStringLiteral("*this");
}
value = isValue.cap(2);
} else {
if (!isPrettyValue.exactMatch(m_local)) {
match = isPrettyValue.match(m_local);
if (!match.hasMatch()) {
qDebug() << "Could not parse:" << m_local;
m_local.clear();
return;
}
symbolAndValue << isPrettyValue.cap(1) << isPrettyValue.cap(2);
value = isPrettyValue.cap(3);
symbolAndValue << match.captured(1) << match.captured(2);
value = match.captured(3);
}
QTreeWidgetItem *item;
......@@ -133,7 +139,8 @@ void LocalsView::addLocal(const QString &vString)
item = new QTreeWidgetItem(this, symbolAndValue);
addArray(item, value.mid(1, value.size() - 2));
} else {
if (isStruct.exactMatch(value)) {
match = isStruct.match(value);
if (match.hasMatch()) {
item = new QTreeWidgetItem(this, symbolAndValue);
addStruct(item, value.mid(1, value.size() - 2));
} else {
......@@ -149,8 +156,8 @@ void LocalsView::addLocal(const QString &vString)
void LocalsView::addStruct(QTreeWidgetItem *parent, const QString &vString)
{
static QRegExp isArray(QStringLiteral("\\{\\.*\\s=\\s.*"));
static QRegExp isStruct(QStringLiteral("\\.*\\s=\\s.*"));
static const QRegularExpression isArray(QStringLiteral("\\A\\{\\.*\\s=\\s.*\\z"));
static const QRegularExpression isStruct(QStringLiteral("\\A\\.*\\s=\\s.*\\z"));
QTreeWidgetItem *item;
QStringList symbolAndValue;
QString subValue;
......@@ -198,10 +205,10 @@ void LocalsView::addStruct(QTreeWidgetItem *parent, const QString &vString)
end++;
}
subValue = vString.mid(start, end - start);
if (isArray.exactMatch(subValue)) {
if (isArray.match(subValue).hasMatch()) {
item = new QTreeWidgetItem(parent, symbolAndValue);
addArray(item, subValue);
} else if (isStruct.exactMatch(subValue)) {
} else if (isStruct.match(subValue).hasMatch()) {
item = new QTreeWidgetItem(parent, symbolAndValue);
addStruct(item, subValue);
} else {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment