diff options
author | Alessandro Portale <alessandro.portale@qt.io> | 2018-11-11 10:08:29 +0100 |
---|---|---|
committer | Alessandro Portale <alessandro.portale@qt.io> | 2018-11-12 11:10:38 +0000 |
commit | 901b2be036618f562ecfc3bf48be58830609bfe7 (patch) | |
tree | e078d729c33efb6c45dfaa67b8d112fb5e33396e /src/plugins/help | |
parent | 644110f6e03eae18ae9c382465a3f7eb550653a6 (diff) | |
download | qt-creator-901b2be036618f562ecfc3bf48be58830609bfe7.tar.gz |
HelpPlugin: Modernize
modernize-use-*
Change-Id: Ie0058675c1336377326089dbc093436130f1fe03
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
Reviewed-by: Eike Ziller <eike.ziller@qt.io>
Diffstat (limited to 'src/plugins/help')
-rw-r--r-- | src/plugins/help/docsettingspage.cpp | 4 | ||||
-rw-r--r-- | src/plugins/help/docsettingspage.h | 2 | ||||
-rw-r--r-- | src/plugins/help/filtersettingspage.h | 2 | ||||
-rw-r--r-- | src/plugins/help/helpindexfilter.cpp | 4 | ||||
-rw-r--r-- | src/plugins/help/helpplugin.cpp | 16 | ||||
-rw-r--r-- | src/plugins/help/helpviewer.cpp | 11 | ||||
-rw-r--r-- | src/plugins/help/helpviewer.h | 2 | ||||
-rw-r--r-- | src/plugins/help/helpwidget.cpp | 14 | ||||
-rw-r--r-- | src/plugins/help/macwebkithelpviewer.h | 54 | ||||
-rw-r--r-- | src/plugins/help/openpagesmodel.cpp | 2 | ||||
-rw-r--r-- | src/plugins/help/openpagesswitcher.cpp | 10 | ||||
-rw-r--r-- | src/plugins/help/openpageswidget.cpp | 4 | ||||
-rw-r--r-- | src/plugins/help/remotehelpfilter.cpp | 6 | ||||
-rw-r--r-- | src/plugins/help/searchwidget.cpp | 30 | ||||
-rw-r--r-- | src/plugins/help/textbrowserhelpviewer.cpp | 16 | ||||
-rw-r--r-- | src/plugins/help/xbelsupport.cpp | 2 |
16 files changed, 81 insertions, 98 deletions
diff --git a/src/plugins/help/docsettingspage.cpp b/src/plugins/help/docsettingspage.cpp index 6ca884ceb2..51a8a7f330 100644 --- a/src/plugins/help/docsettingspage.cpp +++ b/src/plugins/help/docsettingspage.cpp @@ -68,7 +68,7 @@ static DocEntry createEntry(const QString &nameSpace, const QString &fileName, b class DocModel : public QAbstractListModel { public: - typedef QVector<DocEntry> DocEntries; + using DocEntries = QVector<DocEntry>; explicit DocModel(const DocEntries &e = DocEntries(), QObject *parent = nullptr) : QAbstractListModel(parent), m_docEntries(e) {} @@ -268,7 +268,7 @@ bool DocSettingsPage::eventFilter(QObject *object, QEvent *event) return IOptionsPage::eventFilter(object, event); if (event->type() == QEvent::KeyPress) { - QKeyEvent *ke = static_cast<QKeyEvent*>(event); + auto ke = static_cast<const QKeyEvent*>(event); switch (ke->key()) { case Qt::Key_Delete: removeDocumentation(currentSelection()); diff --git a/src/plugins/help/docsettingspage.h b/src/plugins/help/docsettingspage.h index dfe80b6966..f28d8099aa 100644 --- a/src/plugins/help/docsettingspage.h +++ b/src/plugins/help/docsettingspage.h @@ -63,7 +63,7 @@ private: QString m_recentDialogPath; - typedef QHash<QString, QString> NameSpaceToPathHash; + using NameSpaceToPathHash = QHash<QString, QString>; NameSpaceToPathHash m_filesToRegister; QHash<QString, bool> m_filesToRegisterUserManaged; NameSpaceToPathHash m_filesToUnregister; diff --git a/src/plugins/help/filtersettingspage.h b/src/plugins/help/filtersettingspage.h index 1cf43474d7..4d5d102107 100644 --- a/src/plugins/help/filtersettingspage.h +++ b/src/plugins/help/filtersettingspage.h @@ -59,7 +59,7 @@ private: Ui::FilterSettingsPage m_ui; QPointer<QWidget> m_widget; - typedef QMap<QString, QStringList> FilterMap; + using FilterMap = QMap<QString, QStringList>; FilterMap m_filterMap; FilterMap m_filterMapBackup; diff --git a/src/plugins/help/helpindexfilter.cpp b/src/plugins/help/helpindexfilter.cpp index 568b32a03f..e205359cf4 100644 --- a/src/plugins/help/helpindexfilter.cpp +++ b/src/plugins/help/helpindexfilter.cpp @@ -65,9 +65,7 @@ HelpIndexFilter::HelpIndexFilter() this, &HelpIndexFilter::invalidateCache); } -HelpIndexFilter::~HelpIndexFilter() -{ -} +HelpIndexFilter::~HelpIndexFilter() = default; void HelpIndexFilter::prepareSearch(const QString &entry) { diff --git a/src/plugins/help/helpplugin.cpp b/src/plugins/help/helpplugin.cpp index cd04e696fc..78dcad316c 100644 --- a/src/plugins/help/helpplugin.cpp +++ b/src/plugins/help/helpplugin.cpp @@ -203,8 +203,8 @@ HelpPluginPrivate::HelpPluginPrivate() const QString &locale = ICore::userInterfaceLanguage(); if (!locale.isEmpty()) { - QTranslator *qtr = new QTranslator(this); - QTranslator *qhelptr = new QTranslator(this); + auto qtr = new QTranslator(this); + auto qhelptr = new QTranslator(this); const QString &creatorTrPath = ICore::resourcePath() + "/translations"; const QString &qtTrPath = QLibraryInfo::location(QLibraryInfo::TranslationsPath); const QString &trFile = QLatin1String("assistant_") + locale; @@ -397,7 +397,7 @@ void HelpPluginPrivate::saveExternalWindowSettings() HelpWidget *HelpPluginPrivate::createHelpWidget(const Context &context, HelpWidget::WidgetStyle style) { - HelpWidget *widget = new HelpWidget(context, style); + auto widget = new HelpWidget(context, style); connect(widget->currentViewer(), &HelpViewer::loadFinished, this, &HelpPluginPrivate::highlightSearchTermsInContextHelp); @@ -446,8 +446,8 @@ HelpViewer *HelpPluginPrivate::externalHelpViewer() HelpViewer *HelpPlugin::createHelpViewer(qreal zoom) { // check for backends - typedef std::function<HelpViewer *()> ViewerFactory; - typedef QPair<QByteArray, ViewerFactory> ViewerFactoryItem; // id -> factory + using ViewerFactory = std::function<HelpViewer *()>; + using ViewerFactoryItem = QPair<QByteArray, ViewerFactory>; // id -> factory QVector<ViewerFactoryItem> factories; #ifdef QTC_WEBENGINE_HELPVIEWER factories.append(qMakePair(QByteArray("qtwebengine"), []() { return new WebEngineHelpViewer(); })); @@ -491,7 +491,7 @@ HelpViewer *HelpPlugin::createHelpViewer(qreal zoom) viewer->setScale(zoom); // add find support - Aggregation::Aggregate *agg = new Aggregation::Aggregate(); + auto agg = new Aggregation::Aggregate; agg->add(viewer); agg->add(new HelpViewerFindSupport(viewer)); @@ -760,10 +760,10 @@ class DialogClosingOnEscape : public QDialog { public: DialogClosingOnEscape(QWidget *parent = nullptr) : QDialog(parent) {} - bool event(QEvent *event) + bool event(QEvent *event) override { if (event->type() == QEvent::ShortcutOverride) { - QKeyEvent *ke = static_cast<QKeyEvent *>(event); + auto ke = static_cast<QKeyEvent *>(event); if (ke->key() == Qt::Key_Escape && !ke->modifiers()) { ke->accept(); return true; diff --git a/src/plugins/help/helpviewer.cpp b/src/plugins/help/helpviewer.cpp index e2cac11fc9..1f6df7baab 100644 --- a/src/plugins/help/helpviewer.cpp +++ b/src/plugins/help/helpviewer.cpp @@ -78,8 +78,7 @@ static ExtensionMap extensionMap[] = { {".xhtml", "application/xhtml+xml"}, {".wml", "text/vnd.wap.wml"}, {".wmlc", "application/vnd.wap.wmlc"}, - {"about:blank", nullptr}, - {nullptr, nullptr} + {"about:blank", nullptr} }; HelpViewer::HelpViewer(QWidget *parent) @@ -122,11 +121,9 @@ QString HelpViewer::mimeFromUrl(const QUrl &url) const int index = path.lastIndexOf(QLatin1Char('.')); const QByteArray &ext = path.mid(index).toUtf8().toLower(); - const ExtensionMap *e = extensionMap; - while (e->extension) { - if (ext == e->extension) - return QLatin1String(e->mimeType); - ++e; + for (const auto &e : extensionMap) { + if (ext == e.extension) + return QLatin1String(e.mimeType); } return QString(); } diff --git a/src/plugins/help/helpviewer.h b/src/plugins/help/helpviewer.h index d26c1a9f51..9369fd72ca 100644 --- a/src/plugins/help/helpviewer.h +++ b/src/plugins/help/helpviewer.h @@ -111,7 +111,7 @@ protected: void restoreOverrideCursor(); - Actions m_visibleActions = 0; + Actions m_visibleActions; int m_loadOverrideStack = 0; }; diff --git a/src/plugins/help/helpwidget.cpp b/src/plugins/help/helpwidget.cpp index 0e092e0d72..6e75a3d1c7 100644 --- a/src/plugins/help/helpwidget.cpp +++ b/src/plugins/help/helpwidget.cpp @@ -98,19 +98,19 @@ HelpWidget::HelpWidget(const Core::Context &context, WidgetStyle style, QWidget m_sideBarSplitter->setOpaqueResize(false); hLayout->addWidget(m_sideBarSplitter); - Utils::StyledBar *toolBar = new Utils::StyledBar(); - QHBoxLayout *layout = new QHBoxLayout(toolBar); + auto toolBar = new Utils::StyledBar(); + auto layout = new QHBoxLayout(toolBar); layout->setSpacing(0); layout->setMargin(0); auto rightSide = new QWidget(this); m_sideBarSplitter->insertWidget(1, rightSide); - QVBoxLayout *vLayout = new QVBoxLayout(rightSide); + auto vLayout = new QVBoxLayout(rightSide); vLayout->setMargin(0); vLayout->setSpacing(0); vLayout->addWidget(toolBar); vLayout->addWidget(m_viewerStack); - Core::FindToolBarPlaceHolder *fth = new Core::FindToolBarPlaceHolder(this); + auto fth = new Core::FindToolBarPlaceHolder(this); vLayout->addWidget(fth); setFocusProxy(m_viewerStack); @@ -265,7 +265,7 @@ HelpWidget::HelpWidget(const Core::Context &context, WidgetStyle style, QWidget openButton->setPopupMode(QToolButton::InstantPopup); openButton->setProperty("noArrow", true); layout->addWidget(openButton); - QMenu *openMenu = new QMenu(openButton); + auto openMenu = new QMenu(openButton); if (m_switchToHelp) openMenu->addAction(m_switchToHelp); if (style == ModeWidget) { @@ -382,7 +382,7 @@ void HelpWidget::addSideBar() : tr("Ctrl+Shift+/"))); shortcutMap.insert(Constants::HELP_SEARCH, cmd); - Core::SideBarItem *openPagesItem = 0; + Core::SideBarItem *openPagesItem = nullptr; if (m_style == ModeWidget) { QWidget *openPagesWidget = OpenPagesManager::instance().openPagesWidget(); openPagesWidget->setWindowTitle(HelpPlugin::tr(Constants::SB_OPENPAGES)); @@ -693,7 +693,7 @@ void HelpWidget::highlightSearchTerms() { if (m_searchTerms.isEmpty()) return; - HelpViewer *viewer = qobject_cast<HelpViewer *>(sender()); + auto viewer = qobject_cast<HelpViewer *>(sender()); QTC_ASSERT(viewer, return); foreach (const QString& term, m_searchTerms) viewer->findText(term, nullptr, false, true); diff --git a/src/plugins/help/macwebkithelpviewer.h b/src/plugins/help/macwebkithelpviewer.h index d98bc4ddfb..82bc857019 100644 --- a/src/plugins/help/macwebkithelpviewer.h +++ b/src/plugins/help/macwebkithelpviewer.h @@ -46,7 +46,7 @@ class MacWebKitHelpWidget : public QMacCocoaViewContainer public: MacWebKitHelpWidget(MacWebKitHelpViewer *parent); - ~MacWebKitHelpWidget(); + ~MacWebKitHelpWidget() override; WebView *webView() const; void startToolTipTimer(const QPoint &pos, const QString &text); @@ -54,8 +54,8 @@ public: MacWebKitHelpViewer *viewer() const; protected: - void hideEvent(QHideEvent *); - void showEvent(QShowEvent *); + void hideEvent(QHideEvent *) override; + void showEvent(QShowEvent *) override; private: void showToolTip(); @@ -68,44 +68,44 @@ class MacWebKitHelpViewer : public HelpViewer public: explicit MacWebKitHelpViewer(QWidget *parent = nullptr); - ~MacWebKitHelpViewer(); + ~MacWebKitHelpViewer() override; - QFont viewerFont() const; - void setViewerFont(const QFont &font); + QFont viewerFont() const override; + void setViewerFont(const QFont &font) override; - qreal scale() const; - void setScale(qreal scale); + qreal scale() const override; + void setScale(qreal scale) override; - QString title() const; + QString title() const override; - QUrl source() const; - void setSource(const QUrl &url); + QUrl source() const override; + void setSource(const QUrl &url) override; void scrollToAnchor(const QString &anchor); - void highlightId(const QString &id) { Q_UNUSED(id) } + void highlightId(const QString &id) override { Q_UNUSED(id) } - void setHtml(const QString &html); + void setHtml(const QString &html) override; - QString selectedText() const; - bool isForwardAvailable() const; - bool isBackwardAvailable() const; - void addBackHistoryItems(QMenu *backMenu); - void addForwardHistoryItems(QMenu *forwardMenu); + QString selectedText() const override; + bool isForwardAvailable() const override; + bool isBackwardAvailable() const override; + void addBackHistoryItems(QMenu *backMenu) override; + void addForwardHistoryItems(QMenu *forwardMenu) override; void setActionVisible(bool visible); bool findText(const QString &text, Core::FindFlags flags, - bool incremental, bool fromSearch, bool *wrapped = nullptr); + bool incremental, bool fromSearch, bool *wrapped = nullptr) override; MacWebKitHelpWidget *widget() const { return m_widget; } public: - void scaleUp(); - void scaleDown(); - void resetScale(); - void copy(); - void stop(); - void forward(); - void backward(); - void print(QPrinter *printer); + void scaleUp() override; + void scaleDown() override; + void resetScale() override; + void copy() override; + void stop() override; + void forward() override; + void backward() override; + void print(QPrinter *printer) override; void slotLoadStarted(); void slotLoadFinished(); diff --git a/src/plugins/help/openpagesmodel.cpp b/src/plugins/help/openpagesmodel.cpp index 3ae610824e..d00600737e 100644 --- a/src/plugins/help/openpagesmodel.cpp +++ b/src/plugins/help/openpagesmodel.cpp @@ -97,7 +97,7 @@ HelpViewer *OpenPagesModel::pageAt(int index) const void OpenPagesModel::handleTitleChanged() { - HelpViewer *page = static_cast<HelpViewer *>(sender()); + auto page = static_cast<HelpViewer *>(sender()); const int row = m_pages.indexOf(page); Q_ASSERT(row != -1 ); const QModelIndex &item = index(row, 0); diff --git a/src/plugins/help/openpagesswitcher.cpp b/src/plugins/help/openpagesswitcher.cpp index 5f5b80f944..091239c14f 100644 --- a/src/plugins/help/openpagesswitcher.cpp +++ b/src/plugins/help/openpagesswitcher.cpp @@ -58,7 +58,7 @@ OpenPagesSwitcher::OpenPagesSwitcher(OpenPagesModel *model) m_openPagesWidget->allowContextMenu(false); m_openPagesWidget->installEventFilter(this); - QVBoxLayout *layout = new QVBoxLayout(this); + auto layout = new QVBoxLayout(this); layout->setMargin(0); layout->addWidget(m_openPagesWidget); @@ -68,9 +68,7 @@ OpenPagesSwitcher::OpenPagesSwitcher(OpenPagesModel *model) this, &OpenPagesSwitcher::setCurrentPage); } -OpenPagesSwitcher::~OpenPagesSwitcher() -{ -} +OpenPagesSwitcher::~OpenPagesSwitcher() = default; void OpenPagesSwitcher::gotoNextPage() { @@ -110,7 +108,7 @@ bool OpenPagesSwitcher::eventFilter(QObject *object, QEvent *event) { if (object == m_openPagesWidget) { if (event->type() == QEvent::KeyPress) { - QKeyEvent *ke = static_cast<QKeyEvent*>(event); + auto ke = static_cast<const QKeyEvent*>(event); if (ke->key() == Qt::Key_Escape) { setVisible(false); return true; @@ -129,7 +127,7 @@ bool OpenPagesSwitcher::eventFilter(QObject *object, QEvent *event) else if (key == Qt::Key_Tab && (ke->modifiers() == modifier)) gotoPreviousPage(); } else if (event->type() == QEvent::KeyRelease) { - QKeyEvent *ke = static_cast<QKeyEvent*>(event); + auto ke = static_cast<const QKeyEvent*>(event); if (ke->modifiers() == 0 /*HACK this is to overcome some event inconsistencies between platforms*/ || (ke->modifiers() == Qt::AltModifier diff --git a/src/plugins/help/openpageswidget.cpp b/src/plugins/help/openpageswidget.cpp index 0878b03fd8..a6d98bbb70 100644 --- a/src/plugins/help/openpageswidget.cpp +++ b/src/plugins/help/openpageswidget.cpp @@ -59,9 +59,7 @@ OpenPagesWidget::OpenPagesWidget(OpenPagesModel *sourceModel, QWidget *parent) this, &OpenPagesWidget::updateCloseButtonVisibility); } -OpenPagesWidget::~OpenPagesWidget() -{ -} +OpenPagesWidget::~OpenPagesWidget() = default; void OpenPagesWidget::selectCurrentPage() { diff --git a/src/plugins/help/remotehelpfilter.cpp b/src/plugins/help/remotehelpfilter.cpp index e01ff9c994..683754c75d 100644 --- a/src/plugins/help/remotehelpfilter.cpp +++ b/src/plugins/help/remotehelpfilter.cpp @@ -46,7 +46,7 @@ RemoteFilterOptions::RemoteFilterOptions(RemoteHelpFilter *filter, QWidget *pare const QStringList remoteUrls = m_filter->remoteUrls(); for (const QString &url : remoteUrls) { - QListWidgetItem *item = new QListWidgetItem(url); + auto item = new QListWidgetItem(url); m_ui.listWidget->addItem(item); item->setFlags(item->flags() | Qt::ItemIsEditable); } @@ -127,9 +127,7 @@ RemoteHelpFilter::RemoteHelpFilter() m_remoteUrls.append("https://en.wikipedia.org/w/index.php?search=%1"); } -RemoteHelpFilter::~RemoteHelpFilter() -{ -} +RemoteHelpFilter::~RemoteHelpFilter() = default; QList<Core::LocatorFilterEntry> RemoteHelpFilter::matchesFor(QFutureInterface<Core::LocatorFilterEntry> &future, const QString &entry) { diff --git a/src/plugins/help/searchwidget.cpp b/src/plugins/help/searchwidget.cpp index 2f3047064f..ccf120b59a 100644 --- a/src/plugins/help/searchwidget.cpp +++ b/src/plugins/help/searchwidget.cpp @@ -55,17 +55,13 @@ using namespace Help::Internal; -SearchWidget::SearchWidget() -{ -} +SearchWidget::SearchWidget() = default; -SearchWidget::~SearchWidget() -{ -} +SearchWidget::~SearchWidget() = default; void SearchWidget::zoomIn() { - QTextBrowser* browser = resultWidget->findChild<QTextBrowser*>(); + auto browser = resultWidget->findChild<QTextBrowser*>(); if (browser && zoomCount != 10) { zoomCount++; browser->zoomIn(); @@ -74,7 +70,7 @@ void SearchWidget::zoomIn() void SearchWidget::zoomOut() { - QTextBrowser* browser = resultWidget->findChild<QTextBrowser*>(); + auto browser = resultWidget->findChild<QTextBrowser*>(); if (browser && zoomCount != -5) { zoomCount--; browser->zoomOut(); @@ -86,7 +82,7 @@ void SearchWidget::resetZoom() if (zoomCount == 0) return; - QTextBrowser* browser = resultWidget->findChild<QTextBrowser*>(); + auto browser = resultWidget->findChild<QTextBrowser*>(); if (browser) { browser->zoomOut(zoomCount); zoomCount = 0; @@ -102,13 +98,13 @@ void SearchWidget::reindexDocumentation() void SearchWidget::showEvent(QShowEvent *event) { if (!event->spontaneous() && !searchEngine) { - QVBoxLayout *vLayout = new QVBoxLayout(this); + auto vLayout = new QVBoxLayout(this); vLayout->setMargin(0); vLayout->setSpacing(0); searchEngine = new QHelpSearchEngine(&LocalHelpManager::helpEngine(), this); - Utils::StyledBar *toolbar = new Utils::StyledBar(this); + auto toolbar = new Utils::StyledBar(this); toolbar->setSingleRow(false); m_queryWidget = searchEngine->queryWidget(); QLayout *tbLayout = new QVBoxLayout(); @@ -120,7 +116,7 @@ void SearchWidget::showEvent(QShowEvent *event) tbLayout->addWidget(m_indexingDocumentationLabel); toolbar->setLayout(tbLayout); - Utils::StyledBar *toolbar2 = new Utils::StyledBar(this); + auto toolbar2 = new Utils::StyledBar(this); toolbar2->setSingleRow(false); tbLayout = new QVBoxLayout(); tbLayout->setSpacing(0); @@ -149,7 +145,7 @@ void SearchWidget::showEvent(QShowEvent *event) connect(searchEngine, &QHelpSearchEngine::searchingFinished, this, &SearchWidget::searchingFinished); - QTextBrowser* browser = resultWidget->findChild<QTextBrowser*>(); + auto browser = resultWidget->findChild<const QTextBrowser*>(); browser->viewport()->installEventFilter(this); connect(searchEngine, &QHelpSearchEngine::indexingStarted, this, @@ -210,10 +206,10 @@ void SearchWidget::indexingFinished() bool SearchWidget::eventFilter(QObject *o, QEvent *e) { - QTextBrowser *browser = resultWidget->findChild<QTextBrowser *>(); + auto browser = resultWidget->findChild<const QTextBrowser *>(); if (browser && o == browser->viewport() && e->type() == QEvent::MouseButtonRelease){ - QMouseEvent *me = static_cast<QMouseEvent *>(e); + auto me = static_cast<const QMouseEvent *>(e); QUrl link = resultWidget->linkAt(me->pos()); if (!link.isEmpty() || link.isValid()) { bool controlPressed = me->modifiers() & Qt::ControlModifier; @@ -228,7 +224,7 @@ bool SearchWidget::eventFilter(QObject *o, QEvent *e) void SearchWidget::contextMenuEvent(QContextMenuEvent *contextMenuEvent) { - QTextBrowser *browser = resultWidget->findChild<QTextBrowser *>(); + auto browser = resultWidget->findChild<QTextBrowser *>(); if (!browser) return; @@ -280,7 +276,7 @@ SearchSideBarItem::SearchSideBarItem() QList<QToolButton *> SearchSideBarItem::createToolBarWidgets() { - QToolButton *reindexButton = new QToolButton; + auto reindexButton = new QToolButton; reindexButton->setIcon(Utils::Icons::RELOAD.icon()); reindexButton->setToolTip(tr("Regenerate Index")); connect(reindexButton, &QAbstractButton::clicked, diff --git a/src/plugins/help/textbrowserhelpviewer.cpp b/src/plugins/help/textbrowserhelpviewer.cpp index 930df58207..e0f151c30b 100644 --- a/src/plugins/help/textbrowserhelpviewer.cpp +++ b/src/plugins/help/textbrowserhelpviewer.cpp @@ -52,7 +52,7 @@ TextBrowserHelpViewer::TextBrowserHelpViewer(QWidget *parent) , m_textBrowser(new TextBrowserHelpWidget(this)) { m_textBrowser->setOpenLinks(false); - QVBoxLayout *layout = new QVBoxLayout; + auto layout = new QVBoxLayout; setLayout(layout); layout->setContentsMargins(0, 0, 0, 0); layout->addWidget(m_textBrowser, 10); @@ -73,9 +73,7 @@ TextBrowserHelpViewer::TextBrowserHelpViewer(QWidget *parent) connect(m_textBrowser, &QTextBrowser::backwardAvailable, this, &HelpViewer::backwardAvailable); } -TextBrowserHelpViewer::~TextBrowserHelpViewer() -{ -} +TextBrowserHelpViewer::~TextBrowserHelpViewer() = default; QFont TextBrowserHelpViewer::viewerFont() const { @@ -179,7 +177,7 @@ bool TextBrowserHelpViewer::isBackwardAvailable() const void TextBrowserHelpViewer::addBackHistoryItems(QMenu *backMenu) { for (int i = 1; i <= m_textBrowser->backwardHistoryCount(); ++i) { - QAction *action = new QAction(backMenu); + auto action = new QAction(backMenu); action->setText(m_textBrowser->historyTitle(-i)); action->setData(-i); connect(action, &QAction::triggered, this, &TextBrowserHelpViewer::goToHistoryItem); @@ -190,7 +188,7 @@ void TextBrowserHelpViewer::addBackHistoryItems(QMenu *backMenu) void TextBrowserHelpViewer::addForwardHistoryItems(QMenu *forwardMenu) { for (int i = 1; i <= m_textBrowser->forwardHistoryCount(); ++i) { - QAction *action = new QAction(forwardMenu); + auto action = new QAction(forwardMenu); action->setText(m_textBrowser->historyTitle(i)); action->setData(i); connect(action, &QAction::triggered, this, &TextBrowserHelpViewer::goToHistoryItem); @@ -281,7 +279,7 @@ void TextBrowserHelpViewer::print(QPrinter *printer) void TextBrowserHelpViewer::goToHistoryItem() { - QAction *action = qobject_cast<QAction *>(sender()); + auto action = qobject_cast<const QAction *>(sender()); QTC_ASSERT(action, return); bool ok = false; int index = action->data().toInt(&ok); @@ -393,14 +391,14 @@ bool TextBrowserHelpWidget::eventFilter(QObject *obj, QEvent *event) if (!forceFont) return true; } else if (event->type() == QEvent::KeyPress) { - QKeyEvent *keyEvent = static_cast<QKeyEvent *>(event); + auto keyEvent = static_cast<QKeyEvent *>(event); if (keyEvent->key() == Qt::Key_Slash) { keyEvent->accept(); Core::Find::openFindToolBar(Core::Find::FindForwardDirection); return true; } } else if (event->type() == QEvent::ToolTip) { - QHelpEvent *e = static_cast<QHelpEvent *>(event); + auto e = static_cast<const QHelpEvent *>(event); QToolTip::showText(e->globalPos(), linkAt(e->pos())); return true; } diff --git a/src/plugins/help/xbelsupport.cpp b/src/plugins/help/xbelsupport.cpp index ea4313f5bc..bf890c5e6d 100644 --- a/src/plugins/help/xbelsupport.cpp +++ b/src/plugins/help/xbelsupport.cpp @@ -211,7 +211,7 @@ void XbelReader::readBookmark(QStandardItem *item) QStandardItem *XbelReader::createChildItem(QStandardItem *item) { - QStandardItem *childItem = new QStandardItem(); + auto childItem = new QStandardItem; childItem->setEditable(false); if (item) |