summaryrefslogtreecommitdiff
path: root/src/plugins/autotest/testresultdelegate.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'src/plugins/autotest/testresultdelegate.cpp')
-rw-r--r--src/plugins/autotest/testresultdelegate.cpp24
1 files changed, 12 insertions, 12 deletions
diff --git a/src/plugins/autotest/testresultdelegate.cpp b/src/plugins/autotest/testresultdelegate.cpp
index 48de155a39..9b55623d57 100644
--- a/src/plugins/autotest/testresultdelegate.cpp
+++ b/src/plugins/autotest/testresultdelegate.cpp
@@ -50,9 +50,9 @@ void TestResultDelegate::paint(QPainter *painter, const QStyleOptionViewItem &op
painter->fillRect(opt.rect, background);
painter->setPen(foreground);
- LayoutPositions positions(opt, resultFilterModel);
- const TestResult *testResult = resultFilterModel->testResult(index);
- QTC_ASSERT(testResult, painter->restore();return);
+ const LayoutPositions positions(opt, resultFilterModel);
+ const TestResult testResult = resultFilterModel->testResult(index);
+ QTC_ASSERT(testResult.isValid(), painter->restore(); return);
const QWidget *widget = dynamic_cast<const QWidget*>(painter->device());
QWindow *window = widget ? widget->window()->windowHandle() : nullptr;
@@ -69,15 +69,15 @@ void TestResultDelegate::paint(QPainter *painter, const QStyleOptionViewItem &op
painter->drawText(positions.typeAreaLeft(), positions.top() + fm.ascent(), typeStr);
} else {
QPen tmp = painter->pen();
- if (testResult->result() == ResultType::TestStart)
+ if (testResult.result() == ResultType::TestStart)
painter->setPen(opt.palette.mid().color());
else
- painter->setPen(TestResult::colorForType(testResult->result()));
+ painter->setPen(TestResult::colorForType(testResult.result()));
painter->drawText(positions.typeAreaLeft(), positions.top() + fm.ascent(), typeStr);
painter->setPen(tmp);
}
- QString output = testResult->outputString(selected);
+ QString output = testResult.outputString(selected);
if (selected) {
limitTextOutput(output);
@@ -92,12 +92,12 @@ void TestResultDelegate::paint(QPainter *painter, const QStyleOptionViewItem &op
fm.elidedText(output.left(2000), Qt::ElideRight, positions.textAreaWidth()));
}
- const QString file = testResult->fileName().fileName();
+ const QString file = testResult.fileName().fileName();
painter->setClipRect(positions.fileArea());
painter->drawText(positions.fileAreaLeft(), positions.top() + fm.ascent(), file);
- if (testResult->line()) {
- QString line = QString::number(testResult->line());
+ if (testResult.line()) {
+ QString line = QString::number(testResult.line());
painter->setClipRect(positions.lineArea());
painter->drawText(positions.lineAreaLeft(), positions.top() + fm.ascent(), line);
}
@@ -129,9 +129,9 @@ QSize TestResultDelegate::sizeHint(const QStyleOptionViewItem &option, const QMo
s.setWidth(opt.rect.width() - indentation);
if (selected) {
- const TestResult *testResult = resultFilterModel->testResult(index);
- QTC_ASSERT(testResult, return QSize());
- QString output = testResult->outputString(selected);
+ const TestResult testResult = resultFilterModel->testResult(index);
+ QTC_ASSERT(testResult.isValid(), return {});
+ QString output = testResult.outputString(selected);
limitTextOutput(output);
output.replace('\n', QChar::LineSeparator);
recalculateTextLayout(index, output, opt.font, positions.textAreaWidth() - indentation);