diff --git a/src/gui/addnewtorrentdialog.cpp b/src/gui/addnewtorrentdialog.cpp index eee00034c..d12ff566c 100644 --- a/src/gui/addnewtorrentdialog.cpp +++ b/src/gui/addnewtorrentdialog.cpp @@ -68,8 +68,11 @@ const QString KEY_SAVEPATHHISTORY = SETTINGS_KEY("SavePathHistory"); namespace { - //just a shortcut - inline SettingsStorage *settings() { return SettingsStorage::instance(); } + // just a shortcut + inline SettingsStorage *settings() + { + return SettingsStorage::instance(); + } } AddNewTorrentDialog::AddNewTorrentDialog(QWidget *parent) @@ -88,7 +91,7 @@ AddNewTorrentDialog::AddNewTorrentDialog(QWidget *parent) auto session = BitTorrent::Session::instance(); ui->startTorrentCheckBox->setChecked(!session->isAddTorrentPaused()); - ui->comboTTM->blockSignals(true); //the TreeView size isn't correct if the slot does it job at this point + ui->comboTTM->blockSignals(true); // the TreeView size isn't correct if the slot does it job at this point ui->comboTTM->setCurrentIndex(!session->isAutoTMMDisabledByDefault()); ui->comboTTM->blockSignals(false); populateSavePathComboBox(); @@ -181,9 +184,9 @@ void AddNewTorrentDialog::show(QString source, QWidget *parent) if (Utils::Misc::isUrl(source)) { // Launch downloader Net::DownloadHandler *handler = Net::DownloadManager::instance()->downloadUrl(source, true, 10485760 /* 10MB */, true); - connect(handler, SIGNAL(downloadFinished(QString, QString)), dlg, SLOT(handleDownloadFinished(QString, QString))); - connect(handler, SIGNAL(downloadFailed(QString, QString)), dlg, SLOT(handleDownloadFailed(QString, QString))); - connect(handler, SIGNAL(redirectedToMagnet(QString, QString)), dlg, SLOT(handleRedirectedToMagnet(QString, QString))); + connect(handler, SIGNAL(downloadFinished(QString,QString)), dlg, SLOT(handleDownloadFinished(QString,QString))); + connect(handler, SIGNAL(downloadFailed(QString,QString)), dlg, SLOT(handleDownloadFailed(QString,QString))); + connect(handler, SIGNAL(redirectedToMagnet(QString,QString)), dlg, SLOT(handleRedirectedToMagnet(QString,QString))); } else { bool ok = false; @@ -309,7 +312,6 @@ void AddNewTorrentDialog::showEvent(QShowEvent *event) raise(); } - void AddNewTorrentDialog::showAdvancedSettings(bool show) { const int minimumW = minimumWidth(); @@ -319,7 +321,7 @@ void AddNewTorrentDialog::showAdvancedSettings(bool show) ui->settings_group->setVisible(true); ui->infoGroup->setVisible(true); ui->contentTreeView->setVisible(m_hasMetadata); - static_cast(layout())->insertWidget(layout()->indexOf(ui->never_show_cb) + 1, ui->adv_button); + static_cast(layout())->insertWidget(layout()->indexOf(ui->never_show_cb) + 1, ui->adv_button); } else { ui->adv_button->setText(QString::fromUtf8(C_DOWN)); @@ -337,7 +339,7 @@ void AddNewTorrentDialog::saveSavePathHistory() const // Get current history QStringList history = settings()->loadValue(KEY_SAVEPATHHISTORY).toStringList(); QList historyDirs; - foreach(const QString dir, history) + foreach (const QString dir, history) historyDirs << QDir(dir); if (!historyDirs.contains(selectedSavePath)) { // Add save path to history @@ -381,8 +383,8 @@ void AddNewTorrentDialog::updateDiskSpaceLabel() QString size_string = torrent_size ? Utils::Misc::friendlyUnit(torrent_size) : QString(tr("Not Available", "This size is unavailable.")); size_string += " ("; size_string += tr("Free space on disk: %1").arg(Utils::Misc::friendlyUnit(Utils::Fs::freeDiskSpaceOnPath( - ui->savePathComboBox->itemData( - ui->savePathComboBox->currentIndex()).toString()))); + ui->savePathComboBox->itemData( + ui->savePathComboBox->currentIndex()).toString()))); size_string += ")"; ui->size_lbl->setText(size_string); } @@ -392,8 +394,8 @@ void AddNewTorrentDialog::onSavePathChanged(int index) // Toggle default save path setting checkbox visibility ui->defaultSavePathCheckBox->setChecked(false); ui->defaultSavePathCheckBox->setVisible( - QDir(ui->savePathComboBox->itemData(ui->savePathComboBox->currentIndex()).toString()) - != QDir(BitTorrent::Session::instance()->defaultSavePath())); + QDir(ui->savePathComboBox->itemData(ui->savePathComboBox->currentIndex()).toString()) + != QDir(BitTorrent::Session::instance()->defaultSavePath())); // Remember index m_oldIndex = index; @@ -502,7 +504,7 @@ void AddNewTorrentDialog::renameSelectedFile() QStringList path_items; path_items << index.data().toString(); QModelIndex parent = m_contentModel->parent(index); - while(parent.isValid()) { + while (parent.isValid()) { path_items.prepend(parent.data().toString()); parent = m_contentModel->parent(parent); } @@ -577,7 +579,7 @@ void AddNewTorrentDialog::populateSavePathComboBox() ui->savePathComboBox->addItem(Utils::Fs::toNativePath(savePath), savePath); } -void AddNewTorrentDialog::displayContentTreeMenu(const QPoint&) +void AddNewTorrentDialog::displayContentTreeMenu(const QPoint &) { QMenu myFilesLlistMenu; const QModelIndexList selectedRows = ui->contentTreeView->selectionModel()->selectedRows(0); @@ -620,7 +622,7 @@ void AddNewTorrentDialog::displayContentTreeMenu(const QPoint&) void AddNewTorrentDialog::accept() { if (!m_hasMetadata) - disconnect(this, SLOT(updateMetadata(const BitTorrent::TorrentInfo &))); + disconnect(this, SLOT(updateMetadata(const BitTorrent::TorrentInfo&))); BitTorrent::AddTorrentParams params; @@ -720,8 +722,8 @@ void AddNewTorrentDialog::setupTreeview() ui->contentTreeView->setModel(m_contentModel); m_contentDelegate = new PropListDelegate(nullptr); ui->contentTreeView->setItemDelegate(m_contentDelegate); - connect(ui->contentTreeView, SIGNAL(clicked(const QModelIndex &)), ui->contentTreeView, SLOT(edit(const QModelIndex &))); - connect(ui->contentTreeView, SIGNAL(customContextMenuRequested(const QPoint &)), this, SLOT(displayContentTreeMenu(const QPoint &))); + connect(ui->contentTreeView, SIGNAL(clicked(const QModelIndex&)), ui->contentTreeView, SLOT(edit(const QModelIndex&))); + connect(ui->contentTreeView, SIGNAL(customContextMenuRequested(const QPoint&)), this, SLOT(displayContentTreeMenu(const QPoint&))); // List files in torrent m_contentModel->model()->setupModelData(m_torrentInfo); diff --git a/src/gui/properties/peerlistdelegate.h b/src/gui/properties/peerlistdelegate.h index f883123d7..bfd79bcbc 100644 --- a/src/gui/properties/peerlistdelegate.h +++ b/src/gui/properties/peerlistdelegate.h @@ -38,7 +38,8 @@ #include "base/utils/misc.h" #include "base/utils/string.h" -class PeerListDelegate: public QItemDelegate { +class PeerListDelegate: public QItemDelegate +{ Q_OBJECT public: @@ -67,7 +68,7 @@ public: ~PeerListDelegate() {} - void paint(QPainter * painter, const QStyleOptionViewItem & option, const QModelIndex & index) const override + void paint(QPainter *painter, const QStyleOptionViewItem &option, const QModelIndex &index) const override { painter->save(); @@ -75,7 +76,7 @@ public: QStyleOptionViewItem opt = QItemDelegate::setOptions(index, option); QItemDelegate::drawBackground(painter, opt, index); - switch(index.column()) { + switch (index.column()) { case PORT: { opt.displayAlignment = Qt::AlignRight | Qt::AlignVCenter; QItemDelegate::drawDisplay(painter, opt, option.rect, index.data().toString()); @@ -91,7 +92,7 @@ public: } break; case DOWN_SPEED: - case UP_SPEED:{ + case UP_SPEED: { qreal speed = index.data().toDouble(); if (speed <= 0.0) break; @@ -103,7 +104,7 @@ public: case RELEVANCE: { qreal progress = index.data().toDouble(); opt.displayAlignment = Qt::AlignRight | Qt::AlignVCenter; - QItemDelegate::drawDisplay(painter, opt, opt.rect, Utils::String::fromDouble(progress*100.0, 1)+"%"); + QItemDelegate::drawDisplay(painter, opt, opt.rect, Utils::String::fromDouble(progress * 100.0, 1) + "%"); } break; default: @@ -113,7 +114,7 @@ public: painter->restore(); } - QWidget* createEditor(QWidget*, const QStyleOptionViewItem &, const QModelIndex &) const override + QWidget *createEditor(QWidget *, const QStyleOptionViewItem &, const QModelIndex &) const override { // No editor here return nullptr; diff --git a/src/gui/properties/peerlistwidget.cpp b/src/gui/properties/peerlistwidget.cpp index da2f50951..038ecce2f 100644 --- a/src/gui/properties/peerlistwidget.cpp +++ b/src/gui/properties/peerlistwidget.cpp @@ -100,7 +100,7 @@ PeerListWidget::PeerListWidget(PropertiesWidget *parent) m_resolveCountries = Preferences::instance()->resolvePeerCountries(); if (!m_resolveCountries) hideColumn(PeerListDelegate::COUNTRY); - //Ensure that at least one column is visible at all times + // Ensure that at least one column is visible at all times bool atLeastOne = false; for (unsigned int i = 0; i < PeerListDelegate::IP_HIDDEN; i++) { if (!isColumnHidden(i)) { @@ -110,9 +110,9 @@ PeerListWidget::PeerListWidget(PropertiesWidget *parent) } if (!atLeastOne) setColumnHidden(PeerListDelegate::IP, false); - //To also mitigate the above issue, we have to resize each column when - //its size is 0, because explicitly 'showing' the column isn't enough - //in the above scenario. + // To also mitigate the above issue, we have to resize each column when + // its size is 0, because explicitly 'showing' the column isn't enough + // in the above scenario. for (unsigned int i = 0; i < PeerListDelegate::IP_HIDDEN; i++) if ((columnWidth(i) <= 0) && !isColumnHidden(i)) resizeColumnToContents(i); @@ -128,7 +128,7 @@ PeerListWidget::PeerListWidget(PropertiesWidget *parent) updatePeerHostNameResolutionState(); // SIGNAL/SLOT header()->setContextMenuPolicy(Qt::CustomContextMenu); - connect(header(), SIGNAL(customContextMenuRequested(const QPoint &)), this, SLOT(displayToggleColumnsMenu(const QPoint &))); + connect(header(), SIGNAL(customContextMenuRequested(const QPoint&)), this, SLOT(displayToggleColumnsMenu(const QPoint&))); connect(header(), SIGNAL(sectionClicked(int)), SLOT(handleSortColumnChanged(int))); handleSortColumnChanged(header()->sortIndicatorSection()); m_copyHotkey = new QShortcut(QKeySequence::Copy, this, SLOT(copySelectedPeers()), 0, Qt::WidgetShortcut); @@ -148,11 +148,11 @@ PeerListWidget::~PeerListWidget() delete m_resolver; } -void PeerListWidget::displayToggleColumnsMenu(const QPoint&) +void PeerListWidget::displayToggleColumnsMenu(const QPoint &) { QMenu hideshowColumn(this); hideshowColumn.setTitle(tr("Column visibility")); - QList actions; + QList actions; for (int i = 0; i < PeerListDelegate::IP_HIDDEN; ++i) { if ((i == PeerListDelegate::COUNTRY) && !Preferences::instance()->resolvePeerCountries()) { actions.append(nullptr); // keep the index in sync @@ -193,7 +193,7 @@ void PeerListWidget::updatePeerHostNameResolutionState() if (Preferences::instance()->resolvePeerHostNames()) { if (!m_resolver) { m_resolver = new Net::ReverseResolution(this); - connect(m_resolver, SIGNAL(ipResolved(QString, QString)), SLOT(handleResolved(QString, QString))); + connect(m_resolver, SIGNAL(ipResolved(QString,QString)), SLOT(handleResolved(QString,QString))); loadPeers(m_properties->getCurrentTorrent(), true); } } @@ -218,7 +218,7 @@ void PeerListWidget::updatePeerCountryResolutionState() } } -void PeerListWidget::showPeerListMenu(const QPoint&) +void PeerListWidget::showPeerListMenu(const QPoint &) { QMenu menu; bool emptyMenu = true; @@ -362,7 +362,7 @@ void PeerListWidget::loadPeers(BitTorrent::TorrentHandle *const torrent, bool fo // Delete peers that are gone QSetIterator it(oldeersSet); while (it.hasNext()) { - const QString& ip = it.next(); + const QString &ip = it.next(); m_missingFlags.remove(ip); m_peerAddresses.remove(ip); QStandardItem *item = m_peerItems.take(ip); @@ -370,7 +370,7 @@ void PeerListWidget::loadPeers(BitTorrent::TorrentHandle *const torrent, bool fo } } -QStandardItem* PeerListWidget::addPeer(const QString& ip, BitTorrent::TorrentHandle *const torrent, const BitTorrent::PeerInfo &peer) +QStandardItem *PeerListWidget::addPeer(const QString &ip, BitTorrent::TorrentHandle *const torrent, const BitTorrent::PeerInfo &peer) { int row = m_listModel->rowCount(); // Adding Peer to peer list @@ -460,7 +460,7 @@ void PeerListWidget::wheelEvent(QWheelEvent *event) { event->accept(); - if(event->modifiers() & Qt::ShiftModifier) { + if (event->modifiers() & Qt::ShiftModifier) { // Shift + scroll = horizontal scroll QWheelEvent scrollHEvent(event->pos(), event->globalPos(), event->delta(), event->buttons(), event->modifiers(), Qt::Horizontal); QTreeView::wheelEvent(&scrollHEvent); diff --git a/src/gui/properties/peerlistwidget.h b/src/gui/properties/peerlistwidget.h index 6a6de5abb..3d4a8483b 100644 --- a/src/gui/properties/peerlistwidget.h +++ b/src/gui/properties/peerlistwidget.h @@ -77,8 +77,8 @@ public: private slots: void loadSettings(); void saveSettings() const; - void displayToggleColumnsMenu(const QPoint&); - void showPeerListMenu(const QPoint&); + void displayToggleColumnsMenu(const QPoint &); + void showPeerListMenu(const QPoint &); void banSelectedPeers(); void copySelectedPeers(); void handleSortColumnChanged(int col); @@ -90,7 +90,7 @@ private: QStandardItemModel *m_listModel; PeerListDelegate *m_listDelegate; PeerListSortModel *m_proxyModel; - QHash m_peerItems; + QHash m_peerItems; QHash m_peerAddresses; QSet m_missingFlags; QPointer m_resolver; diff --git a/src/gui/properties/proplistdelegate.cpp b/src/gui/properties/proplistdelegate.cpp index cb4e86ce9..fcf498101 100644 --- a/src/gui/properties/proplistdelegate.cpp +++ b/src/gui/properties/proplistdelegate.cpp @@ -45,18 +45,18 @@ #include "proplistdelegate.h" #include "torrentcontentmodelitem.h" -namespace { +namespace +{ QPalette progressBarDisabledPalette() { - auto getPalette = []() - { - QProgressBar bar; - bar.setEnabled(false); - QStyleOptionProgressBar opt; - opt.initFrom(&bar); - return opt.palette; - }; + auto getPalette = []() { + QProgressBar bar; + bar.setEnabled(false); + QStyleOptionProgressBar opt; + opt.initFrom(&bar); + return opt.palette; + }; static QPalette palette = getPalette(); return palette; } @@ -74,7 +74,7 @@ void PropListDelegate::paint(QPainter *painter, const QStyleOptionViewItem &opti QStyleOptionViewItem opt = QItemDelegate::setOptions(index, option); QItemDelegate::drawBackground(painter, opt, index); - switch(index.column()) { + switch (index.column()) { case PCSIZE: case REMAINING: QItemDelegate::drawDisplay(painter, opt, option.rect, Utils::Misc::friendlyUnit(index.data().toLongLong())); @@ -105,28 +105,28 @@ void PropListDelegate::paint(QPainter *painter, const QStyleOptionViewItem &opti // XXX: To avoid having the progress text on the right of the bar QProxyStyle("fusion").drawControl(QStyle::CE_ProgressBar, &newopt, painter, 0); #endif - } - break; + } + break; case PRIORITY: { - QString text = ""; - switch (index.data().toInt()) { - case prio::MIXED: - text = tr("Mixed", "Mixed (priorities"); - break; - case prio::IGNORED: - text = tr("Not downloaded"); - break; - case prio::HIGH: - text = tr("High", "High (priority)"); - break; - case prio::MAXIMUM: - text = tr("Maximum", "Maximum (priority)"); - break; - default: - text = tr("Normal", "Normal (priority)"); - break; - } - QItemDelegate::drawDisplay(painter, opt, option.rect, text); + QString text = ""; + switch (index.data().toInt()) { + case prio::MIXED: + text = tr("Mixed", "Mixed (priorities"); + break; + case prio::IGNORED: + text = tr("Not downloaded"); + break; + case prio::HIGH: + text = tr("High", "High (priority)"); + break; + case prio::MAXIMUM: + text = tr("Maximum", "Maximum (priority)"); + break; + default: + text = tr("Normal", "Normal (priority)"); + break; + } + QItemDelegate::drawDisplay(painter, opt, option.rect, text); } break; default: @@ -138,9 +138,9 @@ void PropListDelegate::paint(QPainter *painter, const QStyleOptionViewItem &opti void PropListDelegate::setEditorData(QWidget *editor, const QModelIndex &index) const { - QComboBox *combobox = static_cast(editor); + QComboBox *combobox = static_cast(editor); // Set combobox index - switch(index.data().toInt()) { + switch (index.data().toInt()) { case prio::IGNORED: combobox->setCurrentIndex(0); break; @@ -169,7 +169,7 @@ QWidget *PropListDelegate::createEditor(QWidget *parent, const QStyleOptionViewI if (index.data().toInt() == prio::MIXED) return 0; - QComboBox* editor = new QComboBox(parent); + QComboBox *editor = new QComboBox(parent); editor->setFocusPolicy(Qt::StrongFocus); editor->addItem(tr("Do not download", "Do not download (priority)")); editor->addItem(tr("Normal", "Normal (priority)")); @@ -180,11 +180,11 @@ QWidget *PropListDelegate::createEditor(QWidget *parent, const QStyleOptionViewI void PropListDelegate::setModelData(QWidget *editor, QAbstractItemModel *model, const QModelIndex &index) const { - QComboBox *combobox = static_cast(editor); + QComboBox *combobox = static_cast(editor); int value = combobox->currentIndex(); qDebug("PropListDelegate: setModelData(%d)", value); - switch(value) { + switch (value) { case 0: model->setData(index, prio::IGNORED); // IGNORED break; diff --git a/src/gui/properties/proplistdelegate.h b/src/gui/properties/proplistdelegate.h index 90b8babe9..2d5e41335 100644 --- a/src/gui/properties/proplistdelegate.h +++ b/src/gui/properties/proplistdelegate.h @@ -49,7 +49,7 @@ enum PropColumn REMAINING }; -class PropListDelegate : public QItemDelegate +class PropListDelegate: public QItemDelegate { Q_OBJECT @@ -58,11 +58,11 @@ public: void paint(QPainter *painter, const QStyleOptionViewItem &option, const QModelIndex &index) const override; void setEditorData(QWidget *editor, const QModelIndex &index) const override; - QWidget *createEditor(QWidget *parent, const QStyleOptionViewItem &/* option */, const QModelIndex &index) const override; + QWidget *createEditor(QWidget *parent, const QStyleOptionViewItem & /* option */, const QModelIndex &index) const override; public slots: void setModelData(QWidget *editor, QAbstractItemModel *model, const QModelIndex &index) const override; - void updateEditorGeometry(QWidget *editor, const QStyleOptionViewItem &option, const QModelIndex &/* index */) const override; + void updateEditorGeometry(QWidget *editor, const QStyleOptionViewItem &option, const QModelIndex & /* index */) const override; signals: void filteredFilesChanged() const; diff --git a/src/gui/search/searchlistdelegate.cpp b/src/gui/search/searchlistdelegate.cpp index 16c79afff..04d030d3e 100644 --- a/src/gui/search/searchlistdelegate.cpp +++ b/src/gui/search/searchlistdelegate.cpp @@ -49,7 +49,7 @@ void SearchListDelegate::paint(QPainter *painter, const QStyleOptionViewItem &op QStyleOptionViewItem opt = QItemDelegate::setOptions(index, option); QItemDelegate::drawBackground(painter, opt, index); - switch(index.column()) { + switch (index.column()) { case SearchSortModel::SIZE: opt.displayAlignment = Qt::AlignRight | Qt::AlignVCenter; QItemDelegate::drawDisplay(painter, opt, option.rect, Utils::Misc::friendlyUnit(index.data().toLongLong())); diff --git a/src/gui/search/searchlistdelegate.h b/src/gui/search/searchlistdelegate.h index 2131d5bfa..1af6e19ce 100644 --- a/src/gui/search/searchlistdelegate.h +++ b/src/gui/search/searchlistdelegate.h @@ -39,7 +39,7 @@ public: explicit SearchListDelegate(QObject *parent); void paint(QPainter *painter, const QStyleOptionViewItem &option, const QModelIndex &index) const override; - QWidget* createEditor(QWidget*, const QStyleOptionViewItem &, const QModelIndex &) const override; + QWidget *createEditor(QWidget *, const QStyleOptionViewItem &, const QModelIndex &) const override; }; #endif