Skip to content

Commit

Permalink
fix warnings caused by connect(nullptr, ...)
Browse files Browse the repository at this point in the history
  • Loading branch information
lievenhey committed Nov 18, 2022
1 parent f282a0f commit 7e57a77
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
3 changes: 2 additions & 1 deletion src/resultsbottomuppage.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,8 @@ ResultsBottomUpPage::ResultsBottomUpPage(FilterAndZoomStack* filterStack, PerfPa
ResultsUtil::setupCostDelegate(m_model, ui->bottomUpTreeView);
ResultsUtil::setupContextMenu(ui->bottomUpTreeView, contextMenu, m_model, filterStack, this);

connect(parser, &PerfParser::bottomUpDataAvailable, this, &ResultsBottomUpPage::setBottomUpResults);
if (parser)
connect(parser, &PerfParser::bottomUpDataAvailable, this, &ResultsBottomUpPage::setBottomUpResults);

ResultsUtil::setupResultsAggregation(ui->costAggregationComboBox);
}
Expand Down
2 changes: 1 addition & 1 deletion src/resultspagediff.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ ResultsPageDiff::ResultsPageDiff(QWidget* parent)
, m_exportMenu(new QMenu(tr("Export"), this))
, m_resultsBottomUpPage(
new ResultsBottomUpPage(m_filterAndZoomStack, nullptr, m_costContextMenu, m_exportMenu, this))
, m_resultsTopDownPage(new ResultsTopDownPage(m_filterAndZoomStack, m_fileA, m_costContextMenu, this))
, m_resultsTopDownPage(new ResultsTopDownPage(m_filterAndZoomStack, nullptr, m_costContextMenu, this))
, m_resultsFlameGraphPage(new ResultsFlameGraphPage(m_filterAndZoomStack, m_fileA, m_exportMenu, this))
, m_resultsCallerCalleePage(new ResultsCallerCalleePage(m_filterAndZoomStack, m_fileA, m_costContextMenu, this))
, m_timeLineWidget(new TimeLineWidget(m_fileA, m_filterMenu, m_filterAndZoomStack, this))
Expand Down
3 changes: 2 additions & 1 deletion src/resultstopdownpage.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,8 @@ ResultsTopDownPage::ResultsTopDownPage(FilterAndZoomStack* filterStack, PerfPars
ResultsUtil::setupCostDelegate(m_model, ui->topDownTreeView);
ResultsUtil::setupContextMenu(ui->topDownTreeView, contextMenu, m_model, filterStack, this);

connect(parser, &PerfParser::topDownDataAvailable, this, &ResultsTopDownPage::setTopDownResults);
if (parser)
connect(parser, &PerfParser::topDownDataAvailable, this, &ResultsTopDownPage::setTopDownResults);

ResultsUtil::setupResultsAggregation(ui->costAggregationComboBox);
}
Expand Down

0 comments on commit 7e57a77

Please sign in to comment.