diff options
Diffstat (limited to 'src/plugins')
74 files changed, 235 insertions, 236 deletions
diff --git a/src/plugins/CommandLineOptions/PlayListOption/playlistoption.cpp b/src/plugins/CommandLineOptions/PlayListOption/playlistoption.cpp index 4b5bba8eb..e3e891f59 100644 --- a/src/plugins/CommandLineOptions/PlayListOption/playlistoption.cpp +++ b/src/plugins/CommandLineOptions/PlayListOption/playlistoption.cpp @@ -79,7 +79,7 @@ QString PlayListOption::executeCommand(int id, const QStringList &args) << helpString(PL_SHUFFLE_TOGGLE) << helpString(PL_STATE); - foreach (QString line, list) + for(const QString &line : qAsConst(list)) out += CommandLineManager::formatHelpString(line) + "\n"; } break; diff --git a/src/plugins/Effect/ladspa/ladspahost.cpp b/src/plugins/Effect/ladspa/ladspahost.cpp index b962f3d81..8688ab756 100644 --- a/src/plugins/Effect/ladspa/ladspahost.cpp +++ b/src/plugins/Effect/ladspa/ladspahost.cpp @@ -56,7 +56,7 @@ LADSPAHost::LADSPAHost(QObject *parent) : QObject(parent) int id = settings.value("id").toInt(); LADSPAPlugin *plugin = nullptr; - foreach(LADSPAPlugin *p, plugins()) + for(LADSPAPlugin *p : plugins()) { if(p->unique_id == id) { @@ -68,7 +68,7 @@ LADSPAHost::LADSPAHost(QObject *parent) : QObject(parent) continue; LADSPAEffect *effect = createEffect(plugin); - foreach (LADSPAControl *c, effect->controls) + for(LADSPAControl *c : qAsConst(effect->controls)) c->value = settings.value(QString("port%1").arg(c->port), c->value).toFloat(); m_effects.append(effect); @@ -92,7 +92,7 @@ LADSPAHost::~LADSPAHost() settings.setValue("id", (quint64)m_effects[i]->plugin->desc->UniqueID); - foreach (LADSPAControl *c, m_effects[i]->controls) + for(const LADSPAControl *c : qAsConst(m_effects[i]->controls)) settings.setValue(QString("port%1").arg(c->port), c->value); settings.endGroup(); @@ -105,7 +105,7 @@ void LADSPAHost::configure(quint32 freq, int chan) m_chan = chan; m_freq = freq; - foreach (LADSPAEffect *e, m_effects) + for(LADSPAEffect *e : qAsConst(m_effects)) { //deactivate effect deactivateEffect(e); @@ -149,7 +149,7 @@ void LADSPAHost::loadModules() } else directories = ladspa_path.split(':'); - foreach(QString dir, directories) + for(const QString &dir : qAsConst(directories)) findModules(dir); } @@ -160,7 +160,7 @@ void LADSPAHost::findModules(const QString &path) dir.setSorting(QDir::Name); QFileInfoList files = dir.entryInfoList((QStringList() << "*.so")); - foreach(QFileInfo file, files) + for(const QFileInfo &file : qAsConst(files)) { void *library = dlopen(qPrintable(file.absoluteFilePath ()), RTLD_LAZY); if (!library) @@ -359,15 +359,15 @@ void LADSPAHost::activateEffect(LADSPAEffect *e) { LADSPA_Handle handle = desc->instantiate(desc, m_freq); - foreach (LADSPAControl *c, e->controls) + for(LADSPAControl *c : qAsConst(e->controls)) { desc->connect_port(handle, c->port, &c->value); } - foreach (int port, e->in_ports) + for(int port : qAsConst(e->in_ports)) { desc->connect_port(handle, port, m_buf[in_at++]); } - foreach (int port, e->out_ports) + for(int port : qAsConst(e->out_ports)) { desc->connect_port(handle, port, m_buf[out_at++]); } @@ -381,7 +381,7 @@ void LADSPAHost::activateEffect(LADSPAEffect *e) void LADSPAHost::deactivateEffect(LADSPAEffect *e) { const LADSPA_Descriptor *desc = e->plugin->desc; - foreach (LADSPA_Handle handle, e->handles) + for(const LADSPA_Handle &handle : qAsConst(e->handles)) { if(desc->deactivate) desc->deactivate(handle); diff --git a/src/plugins/Effect/ladspa/settingsdialog.cpp b/src/plugins/Effect/ladspa/settingsdialog.cpp index ab52723a3..877b3098e 100644 --- a/src/plugins/Effect/ladspa/settingsdialog.cpp +++ b/src/plugins/Effect/ladspa/settingsdialog.cpp @@ -106,7 +106,7 @@ void SettingsDialog::on_configureButton_clicked() LADSPASlider *slider = nullptr; QLabel *label = nullptr; - foreach(LADSPAControl *c, effect->controls) + for(LADSPAControl *c : qAsConst(effect->controls)) { switch ((int) c->type) { @@ -146,6 +146,6 @@ void SettingsDialog::accept() void SettingsDialog::updateRunningPlugins() { m_ui->runningListWidget->clear(); - foreach(LADSPAEffect *e, LADSPAHost::instance()->effects()) + for(LADSPAEffect *e : LADSPAHost::instance()->effects()) m_ui->runningListWidget->addItem(e->plugin->desc->Name); } diff --git a/src/plugins/Engines/mplayer/mplayerengine.cpp b/src/plugins/Engines/mplayer/mplayerengine.cpp index 70c2097b5..90b69cfd5 100644 --- a/src/plugins/Engines/mplayer/mplayerengine.cpp +++ b/src/plugins/Engines/mplayer/mplayerengine.cpp @@ -66,7 +66,7 @@ TrackInfo *MplayerInfo::createTrackInfo(const QString &path) QString str = QString::fromLocal8Bit(mplayer_process.readAll()).trimmed(); TrackInfo *info = new TrackInfo(path); QStringList lines = str.split("\n"); - foreach(QString line, lines) + for(const QString &line : qAsConst(lines)) { if(rx_id_length.indexIn(line) > -1) info->setDuration((qint64) rx_id_length.cap(1).toDouble() * 1000); @@ -124,7 +124,7 @@ bool MplayerEngine::enqueue(InputSource *source) { QStringList filters = MplayerInfo::filters(); bool supports = false; - foreach(QString filter, filters) + for(const QString &filter : qAsConst(filters)) { QRegExp regexp(filter, Qt::CaseInsensitive, QRegExp::Wildcard); supports = regexp.exactMatch(source->path()); @@ -203,9 +203,8 @@ void MplayerEngine::setMuted(bool muted) void MplayerEngine::readStdOut() { - QString line = QString::fromLocal8Bit(m_process->readAll ()).trimmed(); - QStringList lines = line.split("\n"); - foreach(line, lines) + const QStringList lines = QString::fromLocal8Bit(m_process->readAll()).trimmed().split("\n"); + for(const QString &line : lines) { if (rx_av.indexIn(line) > -1) { diff --git a/src/plugins/Engines/mplayer/mplayerenginefactory.cpp b/src/plugins/Engines/mplayer/mplayerenginefactory.cpp index ec20eb867..33493dfa4 100644 --- a/src/plugins/Engines/mplayer/mplayerenginefactory.cpp +++ b/src/plugins/Engines/mplayer/mplayerenginefactory.cpp @@ -44,7 +44,7 @@ EngineProperties MplayerEngineFactory::properties() const bool MplayerEngineFactory::supports(const QString &source) const { QStringList filters = MplayerInfo::filters(); - foreach(QString filter, filters) + for(const QString &filter : qAsConst(filters)) { QRegExp regexp(filter, Qt::CaseInsensitive, QRegExp::Wildcard); if (regexp.exactMatch(source)) diff --git a/src/plugins/Engines/mplayer/mplayermetadatamodel.cpp b/src/plugins/Engines/mplayer/mplayermetadatamodel.cpp index e1592a3ab..1f9aaf823 100644 --- a/src/plugins/Engines/mplayer/mplayermetadatamodel.cpp +++ b/src/plugins/Engines/mplayer/mplayermetadatamodel.cpp @@ -56,7 +56,7 @@ QList<MetaDataItem> MplayerMetaDataModel::extraProperties() const //mplayer std output parsing QRegExp rx_id("^(ID_.*)=(.*)"); QMap<QString, QString> params; - foreach(QString line, lines) + for(const QString &line : qAsConst(lines)) { if(rx_id.indexIn(line.trimmed()) > -1) params.insert(rx_id.cap(1), rx_id.cap(2)); diff --git a/src/plugins/FileDialogs/QmmpFileDialog/qmmpfiledialogimpl.cpp b/src/plugins/FileDialogs/QmmpFileDialog/qmmpfiledialogimpl.cpp index a433ded9e..6e4cd08d9 100644 --- a/src/plugins/FileDialogs/QmmpFileDialog/qmmpfiledialogimpl.cpp +++ b/src/plugins/FileDialogs/QmmpFileDialog/qmmpfiledialogimpl.cpp @@ -116,9 +116,9 @@ QStringList QmmpFileDialogImpl::selectedFiles () } else { - QModelIndexList ml = fileListView->selectionModel()->selectedIndexes(); - foreach(QModelIndex i,ml) - l << m_model->filePath(i); + const QModelIndexList ml = fileListView->selectionModel()->selectedIndexes(); + for(const QModelIndex &i : ml) + l << m_model->filePath(i); } return l; } @@ -233,7 +233,7 @@ void QmmpFileDialogImpl::on_addPushButton_clicked() ml = fileListView->selectionModel()->selectedIndexes(); else ml = treeView->selectionModel()->selectedIndexes(); - foreach(QModelIndex i,ml) + for(const QModelIndex &i : qAsConst(ml)) { if (!l.contains(m_model->filePath(i))) l << m_model->filePath(i); @@ -315,7 +315,7 @@ void QmmpFileDialogImpl::setModeAndMask(const QString& d,FileDialog::Mode m, con void QmmpFileDialogImpl::loadMountedVolumes() { mountPointsListWidget->clear(); - foreach (QStorageInfo i, QStorageInfo::mountedVolumes()) + for(const QStorageInfo &i : QStorageInfo::mountedVolumes()) { if(i.fileSystemType() == "tmpfs") continue; @@ -369,7 +369,7 @@ void QmmpFileDialogImpl::updateSelection () ml = treeView->selectionModel()->selectedIndexes(); QStringList l; QStringList files; - foreach(QModelIndex i,ml) + for(const QModelIndex &i : qAsConst(ml)) { if (!l.contains(m_model->filePath(i).section("/", -1))) { @@ -394,9 +394,9 @@ void QmmpFileDialogImpl::updateSelection () if (m_mode == FileDialog::AddFiles || m_mode == FileDialog::AddFile/* || FileDialog::SaveFile*/) { addPushButton->setEnabled(true); - foreach(str, files) + for(const QString &file : qAsConst(files)) { - if (QFileInfo(str).isDir()) + if (QFileInfo(file).isDir()) { addPushButton->setEnabled(false); break; @@ -445,7 +445,7 @@ void QmmpFileDialogImpl::addFiles(const QStringList &list) //check file extension QString f_name = fileNameLineEdit->text(); bool contains = false; - foreach(QString str, qt_clean_filter_list(fileTypeComboBox->currentText())) + for(const QString &str : qt_clean_filter_list(fileTypeComboBox->currentText())) { QRegExp regExp(str); regExp.setPatternSyntax(QRegExp::Wildcard); diff --git a/src/plugins/FileDialogs/TwoPanelFileDialog/twopanelfiledialogimpl.cpp b/src/plugins/FileDialogs/TwoPanelFileDialog/twopanelfiledialogimpl.cpp index 8ae32f3ac..9acb4a43b 100644 --- a/src/plugins/FileDialogs/TwoPanelFileDialog/twopanelfiledialogimpl.cpp +++ b/src/plugins/FileDialogs/TwoPanelFileDialog/twopanelfiledialogimpl.cpp @@ -105,7 +105,7 @@ QStringList TwoPanelFileDialogImpl::selectedFiles() const } else if(m_mode == FileDialog::AddDir || m_mode == FileDialog::AddDirs) { - foreach (QModelIndex index, m_ui.dirListView->selectionModel()->selectedRows(0)) + for(const QModelIndex &index : m_ui.dirListView->selectionModel()->selectedRows(0)) { if(index.isValid() && (m_dirModel->fileName(index) != "..")) { @@ -115,7 +115,7 @@ QStringList TwoPanelFileDialogImpl::selectedFiles() const } else { - foreach (QListWidgetItem *item, m_ui.fileListWidget->selectedItems()) + for(const QListWidgetItem *item : m_ui.fileListWidget->selectedItems()) { l << item->data(Qt::UserRole).toString(); } @@ -156,7 +156,7 @@ void TwoPanelFileDialogImpl::updateFileSelection () QStringList names; - foreach(QString path, paths) + for(const QString &path : qAsConst(paths)) { QString name = QFileInfo(path).fileName(); if (!names.contains(name)) @@ -182,9 +182,9 @@ void TwoPanelFileDialogImpl::updateFileSelection () if(m_mode == FileDialog::AddFiles || m_mode == FileDialog::AddDirsFiles || m_mode == FileDialog::AddFile || m_mode == FileDialog::PlayDirsFiles) { - foreach(str, paths) + for(const QString &path : qAsConst(paths)) { - if(QFileInfo(str).isFile()) + if(QFileInfo(path).isFile()) { m_ui.playButton->setEnabled(true); break; @@ -382,7 +382,7 @@ void TwoPanelFileDialogImpl::updateFileList(const QString &path) dir.setFilter(QDir::Files | QDir::Hidden | QDir::NoSymLinks); dir.setSorting(QDir::Name); - foreach (QFileInfo info, dir.entryInfoList(m_filters)) + for(const QFileInfo &info : dir.entryInfoList(m_filters)) { QListWidgetItem *item = new QListWidgetItem(info.fileName()); item->setIcon(m_dirModel->iconProvider()->icon(info)); @@ -428,7 +428,7 @@ void TwoPanelFileDialogImpl::addFiles(const QStringList &list, bool play) //check file extension QString f_name = m_ui.fileNameLineEdit->text(); bool contains = false; - foreach(QString str, qt_clean_filter_list(m_ui.fileTypeComboBox->currentText())) + for(const QString &str : qt_clean_filter_list(m_ui.fileTypeComboBox->currentText())) { QRegExp regExp(str); regExp.setPatternSyntax(QRegExp::Wildcard); diff --git a/src/plugins/General/converter/converterdialog.cpp b/src/plugins/General/converter/converterdialog.cpp index fc4916c7d..90de6aca1 100644 --- a/src/plugins/General/converter/converterdialog.cpp +++ b/src/plugins/General/converter/converterdialog.cpp @@ -44,7 +44,7 @@ ConverterDialog::ConverterDialog(QList <PlayListTrack *> tracks, QWidget *paren QStringList paths; MetaDataFormatter formatter("%if(%p&%t,%p - %t,%f) - %l"); - foreach(PlayListTrack *track, tracks) + for(const PlayListTrack *track : qAsConst(tracks)) { //skip streams if(track->duration() == 0 || track->path().contains("://")) @@ -148,7 +148,7 @@ void ConverterDialog::on_stopButton_clicked() if(m_converters.isEmpty()) return; - foreach(Converter *c, m_converters) + for(Converter *c : qAsConst(m_converters)) c->stop(); QThreadPool::globalInstance()->waitForDone(); qDeleteAll(m_converters); @@ -287,7 +287,7 @@ void ConverterDialog::readPresets(const QString &path) dataList.last()[key] = value; } - foreach(QVariantMap data, dataList) + for(QVariantMap data : qAsConst(dataList)) { data["read_only"] = path.startsWith(":/"); QString title = data["name"].toString(); diff --git a/src/plugins/General/copypaste/copypaste.cpp b/src/plugins/General/copypaste/copypaste.cpp index e3401d5aa..c2ef73565 100644 --- a/src/plugins/General/copypaste/copypaste.cpp +++ b/src/plugins/General/copypaste/copypaste.cpp @@ -65,7 +65,7 @@ void CopyPaste::cut() qDebug("%s", Q_FUNC_INFO); qDeleteAll(m_buffer); m_buffer.clear(); - foreach(PlayListTrack *track, m_pl_manager->selectedPlayList()->selectedTracks()) + for(PlayListTrack *track : m_pl_manager->selectedPlayList()->selectedTracks()) { m_buffer.append(new PlayListTrack(*track)); } @@ -77,7 +77,7 @@ void CopyPaste::copy() qDebug("%s", Q_FUNC_INFO); qDeleteAll(m_buffer); m_buffer.clear(); - foreach(PlayListTrack *track, m_pl_manager->selectedPlayList()->selectedTracks()) + for(PlayListTrack *track : m_pl_manager->selectedPlayList()->selectedTracks()) { m_buffer.append(new PlayListTrack(*track)); } @@ -86,7 +86,7 @@ void CopyPaste::copy() void CopyPaste::paste() { qDebug("%s", Q_FUNC_INFO); - foreach(PlayListTrack *track, m_buffer) + for(const PlayListTrack *track : qAsConst( m_buffer)) { m_pl_manager->selectedPlayList()->add(new PlayListTrack(*track)); } diff --git a/src/plugins/General/covermanager/coverwidget.cpp b/src/plugins/General/covermanager/coverwidget.cpp index 87169b376..b41f70293 100644 --- a/src/plugins/General/covermanager/coverwidget.cpp +++ b/src/plugins/General/covermanager/coverwidget.cpp @@ -50,7 +50,7 @@ CoverWidget::CoverWidget(QWidget *parent) //settings QSettings settings(Qmmp::configFile(), QSettings::IniFormat); m_size = settings.value("CoverManager/size", 0).toInt(); - foreach(QAction *a, sizeMenu->actions ()) + for(QAction *a : sizeMenu->actions ()) { a->setCheckable(true); if(a->data().toInt() == m_size) diff --git a/src/plugins/General/fileops/fileops.cpp b/src/plugins/General/fileops/fileops.cpp index 16058fdc9..a6478438b 100644 --- a/src/plugins/General/fileops/fileops.cpp +++ b/src/plugins/General/fileops/fileops.cpp @@ -113,7 +113,7 @@ void FileOps::execAction(int n) QMessageBox::Yes | QMessageBox::No) != QMessageBox::Yes) break; - foreach(PlayListTrack *track, tracks) + for(PlayListTrack *track : qAsConst(tracks)) { if (QFile::exists(track->path()) && QFile::remove(track->path())) model->removeTrack(track); @@ -151,7 +151,7 @@ void FileOps::copy(QList<PlayListTrack *> tracks, const QString &dest, MetaDataF progress.show(); progress.setAutoClose (false); int i = 0; - foreach(PlayListTrack *track, tracks) + for(const PlayListTrack *track : qAsConst(tracks)) { if (!QFile::exists(track->path())) continue; @@ -207,9 +207,9 @@ void FileOps::copy(QList<PlayListTrack *> tracks, const QString &dest, MetaDataF progress.close(); } -void FileOps::rename(QList<PlayListTrack *> tracks, MetaDataFormatter *formatter, PlayListModel *model) +void FileOps::rename(const QList<PlayListTrack *> &tracks, MetaDataFormatter *formatter, PlayListModel *model) { - foreach(PlayListTrack *track, tracks) + for(PlayListTrack *track : qAsConst(tracks)) { if (!QFile::exists(track->path())) //is it file? continue; @@ -232,7 +232,7 @@ void FileOps::rename(QList<PlayListTrack *> tracks, MetaDataFormatter *formatter } } -void FileOps::move(QList<PlayListTrack *> tracks, const QString &dest, MetaDataFormatter *formatter, PlayListModel *model) +void FileOps::move(const QList<PlayListTrack *> &tracks, const QString &dest, MetaDataFormatter *formatter, PlayListModel *model) { QProgressDialog progress(qApp->activeWindow ()); progress.setWindowModality(Qt::WindowModal); @@ -241,7 +241,7 @@ void FileOps::move(QList<PlayListTrack *> tracks, const QString &dest, MetaDataF progress.show(); progress.setAutoClose (false); int i = 0; - foreach(PlayListTrack *track, tracks) + for(PlayListTrack *track : qAsConst(tracks)) { if (!QFile::exists(track->path())) continue; diff --git a/src/plugins/General/fileops/fileops.h b/src/plugins/General/fileops/fileops.h index 0ad1fa27e..6fe500ab0 100644 --- a/src/plugins/General/fileops/fileops.h +++ b/src/plugins/General/fileops/fileops.h @@ -54,9 +54,9 @@ private slots: private: void copy(QList<PlayListTrack*> tracks, const QString &dest, MetaDataFormatter *formatter); - void rename(QList<PlayListTrack*> tracks, MetaDataFormatter *formatter, + void rename(const QList<PlayListTrack *> &tracks, MetaDataFormatter *formatter, PlayListModel *model); - void move(QList<PlayListTrack*> tracks, const QString &dest, MetaDataFormatter *formatter, + void move(const QList<PlayListTrack*> &tracks, const QString &dest, MetaDataFormatter *formatter, PlayListModel *model); QList <int> m_types; diff --git a/src/plugins/General/hal/halplugin.cpp b/src/plugins/General/hal/halplugin.cpp index 5b795d3d5..ca1e0952a 100644 --- a/src/plugins/General/hal/halplugin.cpp +++ b/src/plugins/General/hal/halplugin.cpp @@ -47,7 +47,7 @@ HalPlugin::HalPlugin(QObject *parent) : QObject(parent) m_addFiles = false; //find existing devices QStringList udis = m_manager->findDeviceByCapability("volume"); - foreach(QString udi, udis) + for(const QString &udi : qAsConst(udis)) addDevice(udi); //load remaining settings m_addTracks = settings.value("add_tracks", false).toBool(); @@ -63,7 +63,7 @@ HalPlugin::~HalPlugin() void HalPlugin::removeDevice(const QString &udi) { - foreach(HalDevice *device, m_devices) + for(HalDevice *device : qAsConst(m_devices)) { if (device->udi() == udi) { @@ -78,7 +78,7 @@ void HalPlugin::removeDevice(const QString &udi) void HalPlugin::addDevice(const QString &udi) { - foreach(HalDevice *device, m_devices) //is it already exists? + for(const HalDevice *device : qAsConst(m_devices)) //is it already exists? { if (device->udi() == udi) return; @@ -141,7 +141,7 @@ void HalPlugin::addDevice(const QString &udi) void HalPlugin::updateActions() { // add action for cd audio or mounted volume - foreach(HalDevice *device, m_devices) + for(HalDevice *device : qAsConst(m_devices)) { QStringList caps = device->property("info.capabilities").toStringList(); QString dev_path; @@ -184,7 +184,7 @@ void HalPlugin::updateActions() } } // remove action if device is unmounted/removed - foreach(QAction *action, m_actions->actions ()) + for(QAction *action :m_actions->actions ()) { if (!findDevice(action)) { @@ -205,7 +205,7 @@ void HalPlugin::processAction(QAction *action) QAction *HalPlugin::findAction(const QString &dev_path) { - foreach(QAction *action, m_actions->actions ()) + for(QAction *action : m_actions->actions()) { if (action->data().toString() == dev_path) return action; @@ -215,7 +215,7 @@ QAction *HalPlugin::findAction(const QString &dev_path) HalDevice *HalPlugin::findDevice(QAction *action) { - foreach(HalDevice *device, m_devices) + for(HalDevice *device : qAsConst(m_devices)) { QStringList caps = device->property("info.capabilities").toStringList(); QString dev_path; @@ -239,11 +239,11 @@ void HalPlugin::addPath(const QString &path) { PlayListModel *model = PlayListManager::instance()->selectedPlayList(); - foreach(PlayListItem *item, model->items()) // Is it already exist? + for(const PlayListItem *item : model->items()) // Is it already exist? { if(item->isGroup()) continue; - if (dynamic_cast<PlayListTrack *>(item)->path().startsWith(path)) + if (dynamic_cast<const PlayListTrack *>(item)->path().startsWith(path)) return; } diff --git a/src/plugins/General/hotkey/hotkeydialog.cpp b/src/plugins/General/hotkey/hotkeydialog.cpp index e522acc77..4d0c9ac00 100644 --- a/src/plugins/General/hotkey/hotkeydialog.cpp +++ b/src/plugins/General/hotkey/hotkeydialog.cpp @@ -38,7 +38,7 @@ void HotkeyDialog::keyPressEvent (QKeyEvent *event) { m_key = HotkeyManager::keycodeToKeysym(event->nativeScanCode ()); m_modifiers = event->nativeModifiers (); - foreach(long mask_mod, HotkeyManager::ignModifiersList()) + for(long mask_mod : HotkeyManager::ignModifiersList()) m_modifiers &= ~mask_mod; //remove ignored modifiers (num lock, caps lock, etc) m_ui.keyLineEdit->setText(HotkeyManager::getKeyString(m_key, m_modifiers)); diff --git a/src/plugins/General/hotkey/hotkeymanager_win.cpp b/src/plugins/General/hotkey/hotkeymanager_win.cpp index aa0cdd6b5..950b57438 100644 --- a/src/plugins/General/hotkey/hotkeymanager_win.cpp +++ b/src/plugins/General/hotkey/hotkeymanager_win.cpp @@ -226,7 +226,7 @@ bool HotkeyManager::nativeEventFilter(const QByteArray &eventType, void *message {
SoundCore *core = SoundCore::instance();
MediaPlayer *player = MediaPlayer::instance();
- foreach(Hotkey *hotkey, m_grabbedKeys)
+ for(const Hotkey *hotkey : qAsConst(m_grabbedKeys))
{
if(hotkey->id != m->wParam)
continue;
diff --git a/src/plugins/General/hotkey/hotkeymanager_x11.cpp b/src/plugins/General/hotkey/hotkeymanager_x11.cpp index 26aac352b..6819523c6 100644 --- a/src/plugins/General/hotkey/hotkeymanager_x11.cpp +++ b/src/plugins/General/hotkey/hotkeymanager_x11.cpp @@ -95,7 +95,7 @@ HotkeyManager::HotkeyManager(QObject *parent) : QObject(parent) if (key) { - foreach(long mask_mod, ignModifiersList()) + for(long mask_mod : ignModifiersList()) { Hotkey *hotkey = new Hotkey; hotkey->action = i; @@ -155,7 +155,7 @@ bool HotkeyManager::nativeEventFilter(const QByteArray &eventType, void *message quint32 mod = ke->state; SoundCore *core = SoundCore::instance(); MediaPlayer *player = MediaPlayer::instance(); - foreach(Hotkey *hotkey, m_grabbedKeys) + for(const Hotkey *hotkey : qAsConst(m_grabbedKeys)) { if (hotkey->key != key || hotkey->mod != mod) continue; diff --git a/src/plugins/General/hotkey/settingsdialog.cpp b/src/plugins/General/hotkey/settingsdialog.cpp index 7a5bd612d..1255790e2 100644 --- a/src/plugins/General/hotkey/settingsdialog.cpp +++ b/src/plugins/General/hotkey/settingsdialog.cpp @@ -75,7 +75,7 @@ void SettingsDialog::accept() { QSettings settings(Qmmp::configFile(), QSettings::IniFormat); settings.beginGroup("Hotkey"); - foreach(Hotkey *k, m_hotkeys) + for(const Hotkey *k : qAsConst(m_hotkeys)) { settings.setValue(QString("key_%1").arg(k->action), k->key); settings.setValue(QString("modifiers_%1").arg(k->action), k->mod); @@ -86,16 +86,19 @@ void SettingsDialog::accept() void SettingsDialog::on_tableWidget_itemDoubleClicked (QTableWidgetItem *item) { - Hotkey *k = nullptr; - foreach(k, m_hotkeys) + Hotkey *key = nullptr; + for(Hotkey *k : m_hotkeys) { if (k->action == item->type()) + { + key = k; break; + } } - if (!k) + if (!key) return; - HotkeyDialog *dialog = new HotkeyDialog(k->key, k->mod, this); + HotkeyDialog *dialog = new HotkeyDialog(key->key, key->mod, this); if (item->type() >= QTableWidgetItem::UserType && dialog->exec() == QDialog::Accepted) { @@ -104,8 +107,8 @@ void SettingsDialog::on_tableWidget_itemDoubleClicked (QTableWidgetItem *item) if(keyString.isEmpty() || items.isEmpty() || items.first() == item) { item->setText(keyString); - k->key = dialog->keySym (); - k->mod = dialog->nativeModifiers (); + key->key = dialog->keySym (); + key->mod = dialog->nativeModifiers (); } else QMessageBox::warning(this, tr("Warning"), tr("Key sequence '%1' is already used").arg(keyString)); diff --git a/src/plugins/General/listenbrainz/payloadcache.cpp b/src/plugins/General/listenbrainz/payloadcache.cpp index f9e305c66..f142cbd7e 100644 --- a/src/plugins/General/listenbrainz/payloadcache.cpp +++ b/src/plugins/General/listenbrainz/payloadcache.cpp @@ -129,7 +129,7 @@ void PayloadCache::save(const QList<TrackMetaData> &songs) qWarning("PayloadCache: error %d: %s", file.error(), qPrintable(file.errorString())); return; } - foreach(TrackMetaData m, songs) + for(const TrackMetaData &m : qAsConst(songs)) { file.write(QString("title=%1").arg(m.value(Qmmp::TITLE)).toUtf8() +"\n"); file.write(QString("artist=%1").arg(m.value(Qmmp::ARTIST)).toUtf8() +"\n"); diff --git a/src/plugins/General/mpris/player2object.cpp b/src/plugins/General/mpris/player2object.cpp index 7773d86ae..e1b511be5 100644 --- a/src/plugins/General/mpris/player2object.cpp +++ b/src/plugins/General/mpris/player2object.cpp @@ -313,7 +313,7 @@ void Player2Object::emitPropertiesChanged() syncProperties(); QVariantMap map; - foreach(QByteArray name, changedProps) + for(const QByteArray &name : qAsConst(changedProps)) map.insert(name, m_props.value(name)); QDBusMessage msg = QDBusMessage::createSignal("/org/mpris/MediaPlayer2", diff --git a/src/plugins/General/mpris/root2object.cpp b/src/plugins/General/mpris/root2object.cpp index a7e9bdd50..dfc7b8edb 100644 --- a/src/plugins/General/mpris/root2object.cpp +++ b/src/plugins/General/mpris/root2object.cpp @@ -64,9 +64,9 @@ QString Root2Object::identity() const QStringList Root2Object::supportedMimeTypes() const { QStringList mimeTypes; - foreach(DecoderFactory *factory, Decoder::enabledFactories()) + for(const DecoderFactory *factory : Decoder::enabledFactories()) mimeTypes << factory->properties().contentTypes; - foreach(EngineFactory *factory, AbstractEngine::enabledFactories()) + for(const EngineFactory *factory : AbstractEngine::enabledFactories()) mimeTypes << factory->properties().contentTypes; mimeTypes.removeDuplicates(); return mimeTypes; diff --git a/src/plugins/General/notifier/notifier.cpp b/src/plugins/General/notifier/notifier.cpp index 1ad9e6a19..62b4b5641 100644 --- a/src/plugins/General/notifier/notifier.cpp +++ b/src/plugins/General/notifier/notifier.cpp @@ -138,7 +138,7 @@ void Notifier::showMetaData() data.append(m_core->metaData(Qmmp::TRACK).toUtf8()+"\n"); data.append(QString("%1").arg(m_core->duration()/1000).toUtf8()+"\n"); - foreach(QString path, m_psiTuneFiles) + for(const QString &path : qAsConst(m_psiTuneFiles)) { QDir tuneDir = QFileInfo(path).absoluteDir(); if(!tuneDir.exists()) @@ -170,7 +170,7 @@ void Notifier::removePsiTuneFiles() { if(m_psi) //clear psi notification { - foreach(QString path, m_psiTuneFiles) + for(const QString &path : qAsConst(m_psiTuneFiles)) QFile::remove(path); } } diff --git a/src/plugins/General/notifier/settingsdialog.cpp b/src/plugins/General/notifier/settingsdialog.cpp index af5fcaad3..22dd32334 100644 --- a/src/plugins/General/notifier/settingsdialog.cpp +++ b/src/plugins/General/notifier/settingsdialog.cpp @@ -73,7 +73,7 @@ void SettingsDialog::accept() settings.beginGroup("Notifier"); settings.setValue ("message_delay", m_ui.messageDelaySpinBox->value()); uint pos = PopupWidget::BOTTOMLEFT; - foreach (QPushButton *button, m_buttons.values()) + for(QPushButton *button : m_buttons.values()) { if(button->isChecked()) pos = m_buttons.key(button); diff --git a/src/plugins/General/rdetect/removablehelper.cpp b/src/plugins/General/rdetect/removablehelper.cpp index ad4f37a7a..0e891f5ca 100644 --- a/src/plugins/General/rdetect/removablehelper.cpp +++ b/src/plugins/General/rdetect/removablehelper.cpp @@ -155,7 +155,7 @@ void RemovableHelper::updateActions() }
}
// remove action if device is unmounted/removed
- foreach(QAction *action, m_actions->actions())
+ for(const QAction *action : m_actions->actions())
{
bool found = false;
@@ -188,7 +188,7 @@ void RemovableHelper::updateActions() QAction *RemovableHelper::findAction(const QString &dev_path)
{
- foreach(QAction *action, m_actions->actions())
+ for(const QAction *action : m_actions->actions())
{
if (action->data().toString() == dev_path)
return action;
@@ -200,7 +200,7 @@ void RemovableHelper::addPath(const QString &path) {
PlayListModel *model = PlayListManager::instance()->selectedPlayList();
- foreach(PlayListItem *item, model->items()) // Is it already exist?
+ for(const PlayListItem *item : model->items()) // Is it already exist?
{
if(item->isGroup())
continue;
diff --git a/src/plugins/General/rgscan/rgscandialog.cpp b/src/plugins/General/rgscan/rgscandialog.cpp index 78a5276d4..6faa1a225 100644 --- a/src/plugins/General/rgscan/rgscandialog.cpp +++ b/src/plugins/General/rgscan/rgscandialog.cpp @@ -58,7 +58,7 @@ RGScanDialog::RGScanDialog(QList <PlayListTrack *> tracks, QWidget *parent) : Q QStringList paths; MetaDataFormatter formatter("%if(%p&%t,%p - %t,%f) - %l"); - foreach(PlayListTrack *track, tracks) + for(const PlayListTrack *track : qAsConst(tracks)) { //skip streams if(track->duration() == 0 || track->path().contains("://")) @@ -163,7 +163,7 @@ void RGScanDialog::onScanFinished(QString url) bool stopped = true; - foreach (RGScanner *scanner, m_scanners) + for(const RGScanner *scanner : qAsConst(m_scanners)) { if(scanner->isRunning() || scanner->isPending()) stopped = false; @@ -177,7 +177,7 @@ void RGScanDialog::onScanFinished(QString url) QMultiMap<QString, ReplayGainInfoItem*> itemGroupMap; //items grouped by album //group by album name - foreach (RGScanner *scanner, m_scanners) + for(RGScanner *scanner : qAsConst(m_scanners)) { if(!scanner->hasValues()) continue; @@ -190,7 +190,7 @@ void RGScanDialog::onScanFinished(QString url) itemGroupMap.insert(album, item); } //calculate album peak and gain - foreach (QString album, itemGroupMap.keys()) + for(const QString &album : itemGroupMap.keys()) { QList<ReplayGainInfoItem*> items = itemGroupMap.values(album); GainHandle_t **a = (GainHandle_t **) malloc(items.count()*sizeof(GainHandle_t *)); @@ -202,7 +202,7 @@ void RGScanDialog::onScanFinished(QString url) } double album_gain = GetAlbumGain(a, items.count()); free(a); - foreach (ReplayGainInfoItem *item, items) + for(ReplayGainInfoItem *item : qAsConst(items)) { item->info[Qmmp::REPLAYGAIN_ALBUM_PEAK] = album_peak; item->info[Qmmp::REPLAYGAIN_ALBUM_GAIN] = album_gain; @@ -220,7 +220,7 @@ void RGScanDialog::onScanFinished(QString url) { QString url = m_ui.tableWidget->item(i, 0)->data(Qt::UserRole).toString(); bool found = false; - foreach (ReplayGainInfoItem *item, m_replayGainItemList) + for(const ReplayGainInfoItem *item : qAsConst(m_replayGainItemList)) { if(item->url == url) { @@ -256,7 +256,7 @@ void RGScanDialog::stop() { if(m_scanners.isEmpty()) return; - foreach (RGScanner *scaner, m_scanners) + for(RGScanner *scaner : qAsConst(m_scanners)) { scaner->stop(); } @@ -267,7 +267,7 @@ void RGScanDialog::stop() RGScanner *RGScanDialog::findScannerByUrl(const QString &url) { - foreach (RGScanner *scanner, m_scanners) + for(RGScanner *scanner : qAsConst(m_scanners)) { if(scanner->url() == url) return scanner; @@ -395,7 +395,7 @@ void RGScanDialog::on_writeButton_clicked() qDebug("RGScanDialog: writing ReplayGain values..."); - foreach (ReplayGainInfoItem *item, m_replayGainItemList) + for(ReplayGainInfoItem *item : qAsConst(m_replayGainItemList)) { QString ext = item->url.section(".", -1).toLower(); diff --git a/src/plugins/General/scrobbler/scrobbler.cpp b/src/plugins/General/scrobbler/scrobbler.cpp index cc40e9c3f..510f2e6aa 100644 --- a/src/plugins/General/scrobbler/scrobbler.cpp +++ b/src/plugins/General/scrobbler/scrobbler.cpp @@ -298,7 +298,7 @@ void Scrobbler::submit() params.insert("sk", m_session); QStringList keys = params.keys(); - foreach (QString key, keys) //removes empty keys + for(const QString &key : qAsConst(keys)) //removes empty keys { if(params.value(key).isEmpty() || params.value(key) == "0") params.remove(key); @@ -309,7 +309,7 @@ void Scrobbler::submit() QUrlQuery body(""); QByteArray data; - foreach (QString key, params.keys()) + for(const QString &key : params.keys()) { body.addQueryItem(key, params.value(key)); data.append(key.toUtf8() + params.value(key).toUtf8()); @@ -347,7 +347,7 @@ void Scrobbler::sendNotification(const SongInfo &info) params.insert("method", "track.updateNowPlaying"); params.insert("sk", m_session); - foreach (QString key, params) //removes empty keys + for(const QString &key : qAsConst(params)) //removes empty keys { if(params.value(key).isEmpty()) params.remove(key); @@ -358,7 +358,7 @@ void Scrobbler::sendNotification(const SongInfo &info) QUrlQuery body(""); QByteArray data; - foreach (QString key, params.keys()) + for(const QString &key : params.keys()) { body.addQueryItem(key, params.value(key)); data.append(key.toUtf8() + params.value(key).toUtf8()); @@ -469,7 +469,7 @@ void ScrobblerAuth::checkSession(const QString &session) QUrlQuery body(""); QByteArray data; - foreach (QString key, params.keys()) + for(const QString &key : params.keys()) { body.addQueryItem(key, params.value(key)); data.append(key.toUtf8() + params.value(key).toUtf8()); diff --git a/src/plugins/General/scrobbler/scrobblercache.cpp b/src/plugins/General/scrobbler/scrobblercache.cpp index 05319e2a3..9ede37e67 100644 --- a/src/plugins/General/scrobbler/scrobblercache.cpp +++ b/src/plugins/General/scrobbler/scrobblercache.cpp @@ -129,7 +129,7 @@ void ListenCache::save(const QList<SongInfo> &songs) qWarning("ScrobblerCache: error %d: %s", file.error(), qPrintable(file.errorString())); return; } - foreach(SongInfo m, songs) + for(const SongInfo &m : qAsConst(songs)) { file.write(QString("title=%1").arg(m.value(Qmmp::TITLE)).toUtf8() +"\n"); file.write(QString("artist=%1").arg(m.value(Qmmp::ARTIST)).toUtf8() +"\n"); diff --git a/src/plugins/General/statusicon/qmmptrayicon.cpp b/src/plugins/General/statusicon/qmmptrayicon.cpp index 4c435c64d..50af945db 100644 --- a/src/plugins/General/statusicon/qmmptrayicon.cpp +++ b/src/plugins/General/statusicon/qmmptrayicon.cpp @@ -78,7 +78,7 @@ bool QmmpTrayIcon::hasToolTipEvent() { //checking for XEmbed system tray implementation //only this implementation is able to send QHelpEvent - foreach (QWindow *w, qApp->allWindows()) + for(const QWindow *w : qApp->allWindows()) { if(w->objectName() == "QSystemTrayIconSysWindow") return true; diff --git a/src/plugins/General/streambrowser/streamwindow.cpp b/src/plugins/General/streambrowser/streamwindow.cpp index dc276e872..d668068d4 100644 --- a/src/plugins/General/streambrowser/streamwindow.cpp +++ b/src/plugins/General/streambrowser/streamwindow.cpp @@ -185,7 +185,7 @@ void StreamWindow::on_addPushButton_clicked() if(m_ui->tabWidget->currentIndex() == 0) { QModelIndexList indexes = m_ui->favoritesTableView->selectionModel()->selectedRows(0); - foreach(QModelIndex index, indexes) + for(const QModelIndex &index : qAsConst(indexes)) { QModelIndex source_index = m_favoritesFilterModel->mapToSource(index); tracks << new PlayListTrack(); @@ -198,7 +198,7 @@ void StreamWindow::on_addPushButton_clicked() else { QModelIndexList indexes = m_ui->icecastTableView->selectionModel()->selectedRows(0); - foreach(QModelIndex index, indexes) + for(const QModelIndex &index : qAsConst(indexes)) { QModelIndex source_index = m_iceCastFilterModel->mapToSource(index); tracks << new PlayListTrack(); @@ -235,7 +235,7 @@ void StreamWindow::execFavoritesMenu(const QPoint &pos) void StreamWindow::addToFavorites() { QModelIndexList indexes = m_ui->icecastTableView->selectionModel()->selectedRows(0); - foreach(QModelIndex index, indexes) + for(const QModelIndex &index : qAsConst(indexes)) { QModelIndex source_index = m_iceCastFilterModel->mapToSource(index); int row = source_index.row(); @@ -309,7 +309,7 @@ void StreamWindow::removeFromFavorites() return; QModelIndexList indexes = m_ui->favoritesTableView->selectionModel()->selectedRows(0); QList<int> rows_to_remove; - foreach(QModelIndex index, indexes) + for(const QModelIndex &index : qAsConst(indexes)) { rows_to_remove.append(m_favoritesFilterModel->mapToSource(index).row()); } diff --git a/src/plugins/General/udisks2/udisks2device.cpp b/src/plugins/General/udisks2/udisks2device.cpp index db0e88f85..e30ae16dd 100644 --- a/src/plugins/General/udisks2/udisks2device.cpp +++ b/src/plugins/General/udisks2/udisks2device.cpp @@ -90,7 +90,7 @@ QStringList UDisks2Device::mountPoints() const QList<QVariant> args = reply.arguments(); - foreach (QVariant arg, args) + for(const QVariant &arg : qAsConst(args)) { QByteArrayList list; QDBusArgument a = arg.value<QDBusVariant>().variant().value<QDBusArgument>(); @@ -98,7 +98,7 @@ QStringList UDisks2Device::mountPoints() const continue; a >> list; - foreach (QByteArray p, list) + for(const QByteArray &p : qAsConst(list)) points.append(p); } return points; diff --git a/src/plugins/General/udisks2/udisks2plugin.cpp b/src/plugins/General/udisks2/udisks2plugin.cpp index be73686b3..d249c06b0 100644 --- a/src/plugins/General/udisks2/udisks2plugin.cpp +++ b/src/plugins/General/udisks2/udisks2plugin.cpp @@ -49,7 +49,7 @@ UDisks2Plugin::UDisks2Plugin(QObject *parent) : QObject(parent) m_addFiles = false; //find existing devices QList<QDBusObjectPath> devs = m_manager->findAllDevices(); - foreach(QDBusObjectPath o, devs) + for(const QDBusObjectPath &o : qAsConst(devs)) addDevice(o); //load remaining settings m_addTracks = settings.value("add_tracks", false).toBool(); @@ -65,7 +65,7 @@ UDisks2Plugin::~UDisks2Plugin() void UDisks2Plugin::removeDevice(QDBusObjectPath o) { - foreach(UDisks2Device *device, m_devices) + for(UDisks2Device *device : qAsConst(m_devices)) { if (device->objectPath() == o) { @@ -80,7 +80,7 @@ void UDisks2Plugin::removeDevice(QDBusObjectPath o) void UDisks2Plugin::addDevice(QDBusObjectPath o) { - foreach(UDisks2Device *device, m_devices) //is it already exists? + for(const UDisks2Device *device : qAsConst(m_devices)) //is it already exists? { if (device->objectPath() == o) return; @@ -101,7 +101,7 @@ void UDisks2Plugin::addDevice(QDBusObjectPath o) void UDisks2Plugin::updateActions() { // add action for cd audio or mounted volume - foreach(UDisks2Device *device, m_devices) + for(const UDisks2Device *device : qAsConst(m_devices)) { QString dev_path; if (m_detectCDA && device->isAudio()) //cd audio @@ -157,7 +157,7 @@ void UDisks2Plugin::updateActions() } } // remove action if device is unmounted/removed - foreach(QAction *action, m_actions->actions ()) + for(QAction *action : m_actions->actions()) { if (!findDevice(action)) { @@ -179,7 +179,7 @@ void UDisks2Plugin::processAction(QAction *action) QAction *UDisks2Plugin::findAction(const QString &dev_path) { - foreach(QAction *action, m_actions->actions ()) + for(QAction *action : m_actions->actions()) { if (action->data().toString() == dev_path) return action; @@ -189,7 +189,7 @@ QAction *UDisks2Plugin::findAction(const QString &dev_path) UDisks2Device *UDisks2Plugin::findDevice(QAction *action) { - foreach(UDisks2Device *device, m_devices) + for(UDisks2Device *device : qAsConst(m_devices)) { QString dev_path; if (device->isAudio()) @@ -212,7 +212,7 @@ void UDisks2Plugin::addPath(const QString &path) { PlayListModel *model = PlayListManager::instance()->selectedPlayList(); - foreach(PlayListItem *item, model->items()) // Is it already exist? + for(PlayListItem *item : model->items()) // Is it already exist? { if(item->isGroup()) continue; diff --git a/src/plugins/Input/archive/decoder_archive.cpp b/src/plugins/Input/archive/decoder_archive.cpp index 4ebdea9c6..118cb49aa 100644 --- a/src/plugins/Input/archive/decoder_archive.cpp +++ b/src/plugins/Input/archive/decoder_archive.cpp @@ -60,7 +60,7 @@ bool DecoderArchive::initialize() //is this file supported by qmmp? QList<DecoderFactory *> filtered = Decoder::findByFileExtension(filePath); - foreach (DecoderFactory *f, filtered) + for(DecoderFactory *f : qAsConst(filtered)) { if(f->properties().noInput) filtered.removeAll(f); //remove all factories without streaming input @@ -86,7 +86,7 @@ bool DecoderArchive::initialize() else { //multiple factories, so trying to determine by content - foreach (DecoderFactory *fact, filtered) + for(DecoderFactory *fact : qAsConst(filtered)) { if(fact->canDecode(m_input)) { diff --git a/src/plugins/Input/archive/decoderarchivefactory.cpp b/src/plugins/Input/archive/decoderarchivefactory.cpp index 119e30d83..4d569defd 100644 --- a/src/plugins/Input/archive/decoderarchivefactory.cpp +++ b/src/plugins/Input/archive/decoderarchivefactory.cpp @@ -96,7 +96,7 @@ QList<TrackInfo *> DecoderArchiveFactory::createPlayList(const QString &path, Tr //is this file supported by qmmp? QList<DecoderFactory *> filtered = Decoder::findByFileExtension(filePath); - foreach (DecoderFactory *f, filtered) + for(DecoderFactory *f : qAsConst(filtered)) { if(f->properties().noInput) filtered.removeAll(f); //remove all factories without streaming input diff --git a/src/plugins/Input/cdaudio/decodercdaudiofactory.cpp b/src/plugins/Input/cdaudio/decodercdaudiofactory.cpp index f861b8d22..f9e28d413 100644 --- a/src/plugins/Input/cdaudio/decodercdaudiofactory.cpp +++ b/src/plugins/Input/cdaudio/decodercdaudiofactory.cpp @@ -61,7 +61,7 @@ QList<TrackInfo *> DecoderCDAudioFactory::createPlayList(const QString &path, Tr QString device_path = path; device_path.remove("cdda://"); QList <CDATrack> tracks = DecoderCDAudio::generateTrackList(device_path, parts); - foreach(CDATrack t, tracks) + for(const CDATrack &t : qAsConst(tracks)) { list << new TrackInfo(t.info); } diff --git a/src/plugins/Input/cdaudio/settingsdialog.cpp b/src/plugins/Input/cdaudio/settingsdialog.cpp index f63147214..5d31c4a25 100644 --- a/src/plugins/Input/cdaudio/settingsdialog.cpp +++ b/src/plugins/Input/cdaudio/settingsdialog.cpp @@ -78,6 +78,6 @@ void SettingsDialog::on_clearCacheButton_clicked() QDir dir(Qmmp::configDir()); dir.cd("cddbcache"); QStringList list = dir.entryList(QStringList() << "*", QDir::Files); - foreach(QString name, list) + for(const QString &name : qAsConst(list)) dir.remove(name); } diff --git a/src/plugins/Input/cue/settingsdialog.cpp b/src/plugins/Input/cue/settingsdialog.cpp index c43c59b68..5f6a936b4 100644 --- a/src/plugins/Input/cue/settingsdialog.cpp +++ b/src/plugins/Input/cue/settingsdialog.cpp @@ -33,7 +33,7 @@ SettingsDialog::SettingsDialog(QWidget *parent) ui.setupUi(this); setAttribute(Qt::WA_DeleteOnClose); findCodecs(); - foreach (QTextCodec *codec, codecs) + for(const QTextCodec *codec : qAsConst(codecs)) ui.cueEncComboBox->addItem(codec->name()); #ifdef WITH_ENCA @@ -79,7 +79,7 @@ void SettingsDialog::findCodecs() QMap<QString, QTextCodec *> codecMap; QRegExp iso8859RegExp("ISO[- ]8859-([0-9]+).*"); - foreach (int mib, QTextCodec::availableMibs()) + for(int mib : QTextCodec::availableMibs()) { QTextCodec *codec = QTextCodec::codecForMib(mib); diff --git a/src/plugins/Input/mpeg/settingsdialog.cpp b/src/plugins/Input/mpeg/settingsdialog.cpp index 75ef4f0e8..4b9589488 100644 --- a/src/plugins/Input/mpeg/settingsdialog.cpp +++ b/src/plugins/Input/mpeg/settingsdialog.cpp @@ -29,7 +29,7 @@ SettingsDialog::SettingsDialog(bool using_rusxmms, QWidget *parent) m_ui.setupUi(this); setAttribute(Qt::WA_DeleteOnClose); findCodecs(); - foreach (QTextCodec *codec, codecs) + for(const QTextCodec *codec : qAsConst(codecs)) { m_ui.id3v1EncComboBox->addItem(codec->name()); m_ui.id3v2EncComboBox->addItem(codec->name()); @@ -92,7 +92,7 @@ void SettingsDialog::findCodecs() QMap<QString, QTextCodec *> codecMap; QRegExp iso8859RegExp("ISO[- ]8859-([0-9]+).*"); - foreach (int mib, QTextCodec::availableMibs()) + for(int mib : QTextCodec::availableMibs()) { QTextCodec *codec = QTextCodec::codecForMib(mib); diff --git a/src/plugins/Input/wildmidi/wildmidihelper.cpp b/src/plugins/Input/wildmidi/wildmidihelper.cpp index 812930561..e48ff7de9 100644 --- a/src/plugins/Input/wildmidi/wildmidihelper.cpp +++ b/src/plugins/Input/wildmidi/wildmidihelper.cpp @@ -116,7 +116,7 @@ QStringList WildMidiHelper::configFiles() const QStringList files = QStringList() << "/etc/timidity.cfg" << "/etc/timidity/timidity.cfg" << "/etc/wildmidi/wildmidi.cfg"; - foreach(QString path, files) + for(const QString &path : qAsConst(files)) { if(!QFile::exists(path)) files.removeAll(path); diff --git a/src/plugins/Output/qtmultimedia/outputqtmultimedia.cpp b/src/plugins/Output/qtmultimedia/outputqtmultimedia.cpp index 9bd738e01..39ee1fafb 100644 --- a/src/plugins/Output/qtmultimedia/outputqtmultimedia.cpp +++ b/src/plugins/Output/qtmultimedia/outputqtmultimedia.cpp @@ -86,7 +86,7 @@ bool OutputQtMultimedia::initialize(quint32 freq, ChannelMap map, Qmmp::AudioFor if (!saved_device_name.isEmpty()) { const QList<QAudioDeviceInfo> devices = QAudioDeviceInfo::availableDevices(QAudio::AudioOutput); - foreach (const QAudioDeviceInfo &info, devices) + for(const QAudioDeviceInfo &info : devices) { if (info.deviceName()==saved_device_name) { diff --git a/src/plugins/Output/qtmultimedia/settingsdialog.cpp b/src/plugins/Output/qtmultimedia/settingsdialog.cpp index 7b4fd0600..0a4a4532a 100644 --- a/src/plugins/Output/qtmultimedia/settingsdialog.cpp +++ b/src/plugins/Output/qtmultimedia/settingsdialog.cpp @@ -38,7 +38,7 @@ SettingsDialog::SettingsDialog (QWidget *parent) : QDialog (parent) const QList<QAudioDeviceInfo> devices = QAudioDeviceInfo::availableDevices(QAudio::AudioOutput); int index = 1; - foreach (const QAudioDeviceInfo &info, devices) + for(const QAudioDeviceInfo &info : devices) { const QString device_name = info.deviceName(); ui.deviceComboBox->addItem(device_name); diff --git a/src/plugins/PlayListFormats/m3u/m3uplaylistformat.cpp b/src/plugins/PlayListFormats/m3u/m3uplaylistformat.cpp index f004f5099..46d48087d 100644 --- a/src/plugins/PlayListFormats/m3u/m3uplaylistformat.cpp +++ b/src/plugins/PlayListFormats/m3u/m3uplaylistformat.cpp @@ -35,7 +35,7 @@ const PlayListFormatProperties M3UPlaylistFormat::properties() const QList<PlayListTrack *> M3UPlaylistFormat::decode(const QByteArray &contents) { QList<PlayListTrack*> out; - QStringList splitted = QString::fromUtf8(contents).split("\n"); + const QStringList splitted = QString::fromUtf8(contents).split("\n"); if(splitted.isEmpty()) return out; @@ -44,7 +44,7 @@ QList<PlayListTrack *> M3UPlaylistFormat::decode(const QByteArray &contents) QString artist, title; bool hasExtInf = false; - foreach(QString str, splitted) + for(QString str : qAsConst(splitted)) { str = str.trimmed(); if(str.startsWith("#EXTM3U") || str.isEmpty()) @@ -82,7 +82,7 @@ QByteArray M3UPlaylistFormat::encode(const QList<PlayListTrack*> &contents, cons MetaDataFormatter formatter("%if(%p,%p - %t,%t)%if(%p|%t,,%f)"); QString m3uDir = QFileInfo(path).canonicalPath(); - foreach(PlayListTrack* f, contents) + for(const PlayListTrack *f : qAsConst(contents)) { QString info = "#EXTINF:" + QString::number(f->duration() / 1000) + "," + formatter.format(f); out.append(info); diff --git a/src/plugins/PlayListFormats/pls/plsplaylistformat.cpp b/src/plugins/PlayListFormats/pls/plsplaylistformat.cpp index d2682bad6..d4aced290 100644 --- a/src/plugins/PlayListFormats/pls/plsplaylistformat.cpp +++ b/src/plugins/PlayListFormats/pls/plsplaylistformat.cpp @@ -57,7 +57,7 @@ QList<PlayListTrack *> PLSPlaylistFormat::decode(const QByteArray &contents) int number = 0; bool error = false; - foreach (QString line, splitted) + for(const QString &line : qAsConst(splitted)) { if(fileRegExp.indexIn(line) > -1) { @@ -124,7 +124,7 @@ QByteArray PLSPlaylistFormat::encode(const QList<PlayListTrack *> &contents, con QStringList out; out << QString("[playlist]"); int counter = 1; - foreach(PlayListTrack *f, contents) + for(const PlayListTrack *f : qAsConst(contents)) { QString begin = "File" + QString::number(counter) + "="; out.append(begin + f->path()); diff --git a/src/plugins/PlayListFormats/xspf/xspfplaylistformat.cpp b/src/plugins/PlayListFormats/xspf/xspfplaylistformat.cpp index 40ad3eaca..77c538186 100644 --- a/src/plugins/PlayListFormats/xspf/xspfplaylistformat.cpp +++ b/src/plugins/PlayListFormats/xspf/xspfplaylistformat.cpp @@ -126,7 +126,7 @@ QByteArray XSPFPlaylistFormat::encode(const QList<PlayListTrack*> &files, const xml.writeStartElement("trackList"); int counter = 1; - foreach(PlayListTrack* f, files) + for(PlayListTrack *f : qAsConst(files)) { xml.writeStartElement("track"); diff --git a/src/plugins/Transports/http/httpstreamreader.cpp b/src/plugins/Transports/http/httpstreamreader.cpp index bdf241ac0..1451f3b5b 100644 --- a/src/plugins/Transports/http/httpstreamreader.cpp +++ b/src/plugins/Transports/http/httpstreamreader.cpp @@ -503,8 +503,8 @@ void HttpStreamReader::parseICYMetaData(char *data, qint64 size) } #endif QString str = codec->toUnicode(data).trimmed(); - QStringList list(str.split(";", QString::SkipEmptyParts)); - foreach(QString line, list) + const QStringList list(str.split(";", QString::SkipEmptyParts)); + for(QString line : qAsConst(list)) { if (line.contains("StreamTitle=")) { @@ -536,7 +536,7 @@ void HttpStreamReader::parseICYMetaData(char *data, qint64 size) void HttpStreamReader::sendStreamInfo(QTextCodec *codec) { QHash<QString, QString> info; - foreach (QString key, m_stream.header.keys()) + for(const QString &key : m_stream.header.keys()) { info.insert(key, codec->toUnicode(m_stream.header.value(key))); } diff --git a/src/plugins/Transports/http/settingsdialog.cpp b/src/plugins/Transports/http/settingsdialog.cpp index 4bba7bcff..c8631af02 100644 --- a/src/plugins/Transports/http/settingsdialog.cpp +++ b/src/plugins/Transports/http/settingsdialog.cpp @@ -30,7 +30,7 @@ SettingsDialog::SettingsDialog(QWidget *parent) : QDialog(parent) m_ui.setupUi(this); setAttribute(Qt::WA_DeleteOnClose); findCodecs(); - foreach (QTextCodec *codec, codecs) + for(const QTextCodec *codec : qAsConst(codecs)) m_ui.icyEncodingComboBox->addItem(codec->name()); #ifdef WITH_ENCA size_t n = 0; @@ -80,7 +80,7 @@ void SettingsDialog::findCodecs() QMap<QString, QTextCodec *> codecMap; QRegExp iso8859RegExp("ISO[- ]8859-([0-9]+).*"); - foreach (int mib, QTextCodec::availableMibs()) + for(int mib : QTextCodec::availableMibs()) { QTextCodec *codec = QTextCodec::codecForMib(mib); diff --git a/src/plugins/Ui/qsui/aboutqsuidialog.cpp b/src/plugins/Ui/qsui/aboutqsuidialog.cpp index e2a06be2e..595023d53 100644 --- a/src/plugins/Ui/qsui/aboutqsuidialog.cpp +++ b/src/plugins/Ui/qsui/aboutqsuidialog.cpp @@ -66,7 +66,7 @@ QString AboutQSUIDialog::getStringFromResource(const QString& res_file) paths << res_file + ".txt"; paths << res_file; - foreach(QString path, paths) + for(const QString &path : qAsConst(paths)) { QFile file(path); if (file.open(QIODevice::ReadOnly)) diff --git a/src/plugins/Ui/qsui/actionmanager.cpp b/src/plugins/Ui/qsui/actionmanager.cpp index 52597aad8..8e8e0ca7c 100644 --- a/src/plugins/Ui/qsui/actionmanager.cpp +++ b/src/plugins/Ui/qsui/actionmanager.cpp @@ -198,7 +198,7 @@ void ActionManager::saveStates() void ActionManager::saveActions() { QSettings settings(Qmmp::configFile(), QSettings::IniFormat); - foreach(QAction *action, m_actions.values()) + for(const QAction *action : m_actions.values()) { settings.setValue(QString("SimpleUiShortcuts/")+action->objectName(), action->shortcut()); } @@ -206,7 +206,7 @@ void ActionManager::saveActions() void ActionManager::resetShortcuts() { - foreach (QAction *action, m_actions.values()) + for(QAction *action : m_actions.values()) { action->setShortcut(action->property("defaultShortcut").toString()); } @@ -254,7 +254,7 @@ void ActionManager::updateToolBar(QToolBar *toolBar, ActionManager::ToolBarInfo { toolBar->clear(); toolBar->setIconSize(info.iconSize); - foreach (QString name, info.actionNames) + for(const QString &name : qAsConst(info.actionNames)) { if(name == "separator") { @@ -276,7 +276,7 @@ ActionManager::ToolBarInfo ActionManager::defaultToolBar() const idList << PL_ADD_FILE << PL_ADD_DIRECTORY << PREVIOUS << PLAY << PAUSE << STOP << NEXT << EJECT; idList << UI_SEPARATOR << UI_POS_SLIDER << UI_SEPARATOR << UI_VOL_SLIDER << VOL_MUTE; QStringList names; - foreach (Type id, idList) + for(const Type &id : qAsConst(idList)) { if(id == UI_SEPARATOR) { diff --git a/src/plugins/Ui/qsui/equalizer.cpp b/src/plugins/Ui/qsui/equalizer.cpp index cb32a202c..fe1e093d4 100644 --- a/src/plugins/Ui/qsui/equalizer.cpp +++ b/src/plugins/Ui/qsui/equalizer.cpp @@ -113,7 +113,7 @@ Equalizer::Equalizer(QWidget *parent) : QDialog(parent) m_layout->addItem(slidersLayout); m_layout->addItem(buttonsLayout); setMinimumHeight(300); - foreach(QSlider *slider, m_sliders) + for(const QSlider *slider : qAsConst(m_sliders)) { connect(slider, SIGNAL(valueChanged(int)), SLOT(applySettings())); } @@ -181,7 +181,7 @@ void Equalizer::applySettings() void Equalizer::resetSettings() { - foreach(QSlider *slider, m_sliders) + for(QSlider *slider : qAsConst(m_sliders)) { slider->setValue(0); } diff --git a/src/plugins/Ui/qsui/filesystembrowser.cpp b/src/plugins/Ui/qsui/filesystembrowser.cpp index 099b85e62..ae7a07da6 100644 --- a/src/plugins/Ui/qsui/filesystembrowser.cpp +++ b/src/plugins/Ui/qsui/filesystembrowser.cpp @@ -150,7 +150,7 @@ void FileSystemBrowser::onListViewActivated(const QModelIndex &index) void FileSystemBrowser::addToPlayList() { - foreach (QModelIndex index, m_listView->selectionModel()->selectedIndexes()) + for(const QModelIndex &index : m_listView->selectionModel()->selectedIndexes()) { if(!index.isValid()) continue; diff --git a/src/plugins/Ui/qsui/logo.cpp b/src/plugins/Ui/qsui/logo.cpp index 08b98d657..b78caa4c4 100644 --- a/src/plugins/Ui/qsui/logo.cpp +++ b/src/plugins/Ui/qsui/logo.cpp @@ -158,7 +158,7 @@ void Logo::processPreset2() QString str = QString("..0000..");//.arg(Qmmp::strVersion().left(5)); int at = m_value % str.size(); - foreach(QString line, m_source_lines) + for(QString line : qAsConst(m_source_lines)) { while(line.contains("X")) { @@ -178,7 +178,7 @@ void Logo::processPreset3() QString str = QString("...%1...").arg(Qmmp::strVersion().left(5)); int at = m_value % str.size(); - foreach(QString line, m_source_lines) + for(QString line : qAsConst(m_source_lines)) { while(line.contains("X")) { @@ -211,7 +211,7 @@ void Logo::processPreset4() int at = 0; - foreach(QString line, m_source_lines) + for(QString line : qAsConst(m_source_lines)) { int count = line.count("X"); int k = 0; diff --git a/src/plugins/Ui/qsui/mainwindow.cpp b/src/plugins/Ui/qsui/mainwindow.cpp index f7aa07a0a..73891daf3 100644 --- a/src/plugins/Ui/qsui/mainwindow.cpp +++ b/src/plugins/Ui/qsui/mainwindow.cpp @@ -91,7 +91,7 @@ MainWindow::MainWindow(QWidget *parent) : QMainWindow(parent) //keyboard manager m_key_manager = new KeyboardManager(this); //create tabs - foreach(PlayListModel *model, m_pl_manager->playLists()) + for(PlayListModel *model : m_pl_manager->playLists()) { if(m_pl_manager->currentPlayList() != model) m_tabWidget->addTab(model->name()); @@ -692,13 +692,13 @@ void MainWindow::readSettings() QList<QToolBar *> toolBars = findChildren<QToolBar*>(); //clear toolbars to avoid conflicts - foreach (QToolBar *toolBar, toolBars) + for(QToolBar *toolBar : qAsConst(toolBars)) toolBar->clear(); - foreach (ActionManager::ToolBarInfo info, toolBarInfoList) + for(const ActionManager::ToolBarInfo &info : qAsConst(toolBarInfoList)) { bool found = false; - foreach (QToolBar *toolBar, toolBars) + for(QToolBar *toolBar : qAsConst(toolBars)) { if(toolBar->property("uid").toString() == info.uid) { @@ -715,7 +715,7 @@ void MainWindow::readSettings() } } - foreach (QToolBar *toolBar, toolBars) + for(QToolBar *toolBar : qAsConst(toolBars)) toolBar->deleteLater(); toolBars.clear(); setToolBarsBlocked(ACTION(ActionManager::UI_BLOCK_TOOLBARS)->isChecked()); @@ -925,7 +925,7 @@ void MainWindow::setTitleBarsVisible(bool visible) void MainWindow::setToolBarsBlocked(bool blocked) { - foreach (QToolBar *t, findChildren<QToolBar *>()) + for(QToolBar *t : findChildren<QToolBar *>()) { t->setMovable(!blocked); } diff --git a/src/plugins/Ui/qsui/playlistbrowser.cpp b/src/plugins/Ui/qsui/playlistbrowser.cpp index fbe370130..9d831e400 100644 --- a/src/plugins/Ui/qsui/playlistbrowser.cpp +++ b/src/plugins/Ui/qsui/playlistbrowser.cpp @@ -74,7 +74,7 @@ void PlayListBrowser::updateList() { m_listView->selectionModel()->blockSignals(true); m_listModel->clear(); - foreach(PlayListModel *model, m_pl_manager->playLists()) + for(PlayListModel *model : m_pl_manager->playLists()) { QStandardItem *item = new QStandardItem(model->name()); if(m_pl_manager->currentPlayList() == model) diff --git a/src/plugins/Ui/qsui/playlistheader.cpp b/src/plugins/Ui/qsui/playlistheader.cpp index d2b52caa7..2c4bdb9a9 100644 --- a/src/plugins/Ui/qsui/playlistheader.cpp +++ b/src/plugins/Ui/qsui/playlistheader.cpp @@ -79,7 +79,7 @@ PlayListHeader::PlayListHeader(QWidget *parent) : m_alignmentMenu->addAction(tr("Center", "alignment"))->setData(ListWidgetRow::ALIGN_CENTER); connect(m_alignmentMenu, SIGNAL(triggered(QAction*)), SLOT(setAlignment(QAction*))); QActionGroup *alignmentGroup = new QActionGroup(this); - foreach (QAction *a, m_alignmentMenu->actions()) + for(QAction *a : m_alignmentMenu->actions()) { a->setCheckable(true); alignmentGroup->addAction(a); @@ -275,7 +275,7 @@ int PlayListHeader::maxScrollValue() const return 0; int row_width = 0; - foreach (int size, sizes()) + for(int size : sizes()) { row_width += size; } @@ -633,7 +633,7 @@ void PlayListHeader::contextMenuEvent(QContextMenuEvent *e) m_autoResizeAction->setChecked(m_model->data(m_pressed_column, AUTO_RESIZE).toBool()); int alignment = m_model->data(m_pressed_column, ALIGNMENT).toInt(); - foreach (QAction *action, m_alignmentMenu->actions()) + for(QAction *action : m_alignmentMenu->actions()) { if(action->data().toInt() == alignment) { @@ -643,7 +643,7 @@ void PlayListHeader::contextMenuEvent(QContextMenuEvent *e) } //hide unused actions - foreach (QAction *action, m_menu->actions()) + for(QAction *action : m_menu->actions()) { if(m_menu->actions().at(0) == action) action->setVisible(m_model->count() < MAX_COLUMNS); @@ -656,7 +656,7 @@ void PlayListHeader::contextMenuEvent(QContextMenuEvent *e) } else { - foreach (QAction *action, m_menu->actions()) + for(QAction *action : m_menu->actions()) { if(action == m_menu->actions().first()) action->setVisible(m_model->count() < MAX_COLUMNS); diff --git a/src/plugins/Ui/qsui/qsuianalyzer.cpp b/src/plugins/Ui/qsui/qsuianalyzer.cpp index e86174d16..868a042c7 100644 --- a/src/plugins/Ui/qsui/qsuianalyzer.cpp +++ b/src/plugins/Ui/qsui/qsuianalyzer.cpp @@ -238,7 +238,7 @@ void QSUiAnalyzer::createMenu() m_fpsGroup->addAction(tr("25 fps"))->setData(25); m_fpsGroup->addAction(tr("10 fps"))->setData(10); m_fpsGroup->addAction(tr("5 fps"))->setData(5); - foreach(QAction *act, m_fpsGroup->actions ()) + for(QAction *act : m_fpsGroup->actions ()) { act->setCheckable(true); refreshRate->addAction(act); @@ -252,7 +252,7 @@ void QSUiAnalyzer::createMenu() m_analyzerFalloffGroup->addAction(tr("Medium"))->setData(2.2); m_analyzerFalloffGroup->addAction(tr("Fast"))->setData(2.4); m_analyzerFalloffGroup->addAction(tr("Fastest"))->setData(2.8); - foreach(QAction *act, m_analyzerFalloffGroup->actions ()) + for(QAction *act : m_analyzerFalloffGroup->actions ()) { act->setCheckable(true); analyzerFalloff->addAction(act); @@ -266,7 +266,7 @@ void QSUiAnalyzer::createMenu() m_peaksFalloffGroup->addAction(tr("Medium"))->setData(0.2); m_peaksFalloffGroup->addAction(tr("Fast"))->setData(0.4); m_peaksFalloffGroup->addAction(tr("Fastest"))->setData(0.8); - foreach(QAction *act, m_peaksFalloffGroup->actions ()) + for(QAction *act : m_peaksFalloffGroup->actions ()) { act->setCheckable(true); peaksFalloff->addAction(act); @@ -317,17 +317,17 @@ void QSUiAnalyzer::readSettings() m_coverAction->setChecked(m_show_cover); m_peaksAction->setChecked(m_show_peaks); - foreach(QAction *act, m_fpsGroup->actions ()) + for(QAction *act : m_fpsGroup->actions ()) { if (m_timer->interval() == 1000 / act->data().toInt()) act->setChecked(true); } - foreach(QAction *act, m_peaksFalloffGroup->actions ()) + for(QAction *act : m_peaksFalloffGroup->actions ()) { if (m_peaks_falloff == act->data().toDouble()) act->setChecked(true); } - foreach(QAction *act, m_analyzerFalloffGroup->actions ()) + for(QAction *act : m_analyzerFalloffGroup->actions ()) { if (m_analyzer_falloff == act->data().toDouble()) act->setChecked(true); diff --git a/src/plugins/Ui/qsui/shortcutdialog.cpp b/src/plugins/Ui/qsui/shortcutdialog.cpp index 51b357e0f..5d67feaa8 100644 --- a/src/plugins/Ui/qsui/shortcutdialog.cpp +++ b/src/plugins/Ui/qsui/shortcutdialog.cpp @@ -28,7 +28,7 @@ ShortcutDialog::ShortcutDialog(const QString &key, QWidget *parent) m_ui.keyLineEdit->setText(key); //buttons should not catch keys - foreach(QAbstractButton *button, m_ui.buttonBox->buttons()) + for(QAbstractButton *button : m_ui.buttonBox->buttons()) button->setFocusPolicy(Qt::NoFocus); } diff --git a/src/plugins/Ui/qsui/toolbareditor.cpp b/src/plugins/Ui/qsui/toolbareditor.cpp index 4d54e7206..a0024e864 100644 --- a/src/plugins/Ui/qsui/toolbareditor.cpp +++ b/src/plugins/Ui/qsui/toolbareditor.cpp @@ -76,7 +76,7 @@ void ToolBarEditor::populateActionList(bool reset) } QStringList actionNames; - foreach (ActionManager::ToolBarInfo info, m_toolBarInfoList) + for(const ActionManager::ToolBarInfo &info : qAsConst(m_toolBarInfoList)) { actionNames << info.actionNames; m_ui->toolbarNameComboBox->addItem(info.title); @@ -191,7 +191,7 @@ void ToolBarEditor::on_toolbarNameComboBox_activated(int index) return; ActionManager::ToolBarInfo info = m_toolBarInfoList.at(index); - foreach (QString name, info.actionNames) + for(const QString &name : qAsConst(info.actionNames)) { if(name == "separator") { diff --git a/src/plugins/Ui/qsui/visualmenu.cpp b/src/plugins/Ui/qsui/visualmenu.cpp index 0defc96fe..b8f56dc39 100644 --- a/src/plugins/Ui/qsui/visualmenu.cpp +++ b/src/plugins/Ui/qsui/visualmenu.cpp @@ -26,8 +26,7 @@ VisualMenu::VisualMenu(QWidget *parent) : QMenu(tr("Visualization"), parent) { - VisualFactory *factory = nullptr; - foreach(factory, Visual::factories()) + for(VisualFactory *factory : Visual::factories()) { VisualAction *act = new VisualAction(factory, this); addAction(act); diff --git a/src/plugins/Ui/skinned/actionmanager.cpp b/src/plugins/Ui/skinned/actionmanager.cpp index 8ee1fb667..7048ca58f 100644 --- a/src/plugins/Ui/skinned/actionmanager.cpp +++ b/src/plugins/Ui/skinned/actionmanager.cpp @@ -179,7 +179,7 @@ void ActionManager::saveStates() void ActionManager::saveActions() { QSettings settings(Qmmp::configFile(), QSettings::IniFormat); - foreach(QAction *action, m_actions.values()) + for(const QAction *action : m_actions.values()) { settings.setValue(QString("SkinnedShortcuts/")+action->objectName(), action->shortcut()); } @@ -187,7 +187,7 @@ void ActionManager::saveActions() void ActionManager::resetShortcuts() { - foreach (QAction *action, m_actions.values()) + for(QAction *action : m_actions.values()) { action->setShortcut(action->property("defaultShortcut").toString()); } diff --git a/src/plugins/Ui/skinned/dock.cpp b/src/plugins/Ui/skinned/dock.cpp index 7f8c291f1..4da50d4c2 100644 --- a/src/plugins/Ui/skinned/dock.cpp +++ b/src/plugins/Ui/skinned/dock.cpp @@ -199,7 +199,7 @@ void Dock::move (QWidget* mv, QPoint npos) void Dock::calculateDistances() { m_delta_list.clear(); - foreach (QWidget *w, m_widgetList) + for(const QWidget *w : qAsConst(m_widgetList)) { if (w == m_mainWidget) m_delta_list.append(QPoint(0,0)); diff --git a/src/plugins/Ui/skinned/eqwidget.cpp b/src/plugins/Ui/skinned/eqwidget.cpp index 55ff6d797..2cd1efc7e 100644 --- a/src/plugins/Ui/skinned/eqwidget.cpp +++ b/src/plugins/Ui/skinned/eqwidget.cpp @@ -151,7 +151,7 @@ void EqWidget::readSettings() QPoint pos = settings.value("Skinned/eq_pos", QPoint(100, 216)).toPoint(); int r = m_skin->ratio(); //TODO QGuiApplication::screenAt - foreach(QScreen *screen, QGuiApplication::screens()) + for(const QScreen *screen : QGuiApplication::screens()) { if(screen->availableGeometry().contains(pos)) { @@ -321,7 +321,7 @@ void EqWidget::savePreset() preset->setPreamp(m_preamp->value()); for (int i = 0; i<10; ++i) preset->setGain(i, m_sliders.at (i)->value()); - foreach(EQPreset *p, m_presets) //remove preset with same name + for(EQPreset *p : qAsConst(m_presets)) //remove preset with same name { if (p->text() == text) { @@ -391,7 +391,7 @@ void EqWidget::loadPreset(const QString &name) EQPreset *EqWidget::findPreset(const QString &name) { - foreach(EQPreset *preset, m_autoPresets) + for(EQPreset *preset : qAsConst(m_autoPresets)) { if (preset->text() == name) return preset; diff --git a/src/plugins/Ui/skinned/mainvisual.cpp b/src/plugins/Ui/skinned/mainvisual.cpp index 6f24a432f..e726866d9 100644 --- a/src/plugins/Ui/skinned/mainvisual.cpp +++ b/src/plugins/Ui/skinned/mainvisual.cpp @@ -132,7 +132,7 @@ void MainVisual::mousePressEvent (QMouseEvent *e) setVisual(nullptr); QString str = m_vis ? m_vis->name() : "Off"; - foreach(QAction *act, m_visModeGroup->actions ()) + for(QAction *act : m_visModeGroup->actions()) { if (str == act->data().toString()) { @@ -216,7 +216,7 @@ void MainVisual::createMenu() m_visModeGroup->addAction(tr("Analyzer"))->setData("Analyzer"); m_visModeGroup->addAction(tr("Scope"))->setData("Scope"); m_visModeGroup->addAction(tr("Off"))->setData("Off"); - foreach(QAction *act, m_visModeGroup->actions ()) + for(QAction *act : m_visModeGroup->actions()) { act->setCheckable(true); visMode->addAction(act); @@ -230,13 +230,13 @@ void MainVisual::createMenu() m_analyzerModeGroup->addAction(tr("Vertical Lines"))->setData(2); m_analyzerTypeGroup->addAction(tr("Lines"))->setData(0); m_analyzerTypeGroup->addAction(tr("Bars"))->setData(1); - foreach(QAction *act, m_analyzerModeGroup->actions ()) + for(QAction *act : m_analyzerModeGroup->actions()) { act->setCheckable(true); analyzerMode->addAction(act); } analyzerMode->addSeparator (); - foreach(QAction *act, m_analyzerTypeGroup->actions ()) + for(QAction *act : m_analyzerTypeGroup->actions()) { act->setCheckable(true); analyzerMode->addAction(act); @@ -253,7 +253,7 @@ void MainVisual::createMenu() m_fpsGroup->addAction(tr("25 fps"))->setData(25); m_fpsGroup->addAction(tr("10 fps"))->setData(10); m_fpsGroup->addAction(tr("5 fps"))->setData(5); - foreach(QAction *act, m_fpsGroup->actions ()) + for(QAction *act : m_fpsGroup->actions()) { act->setCheckable(true); refreshRate->addAction(act); @@ -267,7 +267,7 @@ void MainVisual::createMenu() m_analyzerFalloffGroup->addAction(tr("Medium"))->setData(2.2); m_analyzerFalloffGroup->addAction(tr("Fast"))->setData(2.4); m_analyzerFalloffGroup->addAction(tr("Fastest"))->setData(2.8); - foreach(QAction *act, m_analyzerFalloffGroup->actions ()) + for(QAction *act : m_analyzerFalloffGroup->actions()) { act->setCheckable(true); analyzerFalloff->addAction(act); @@ -281,7 +281,7 @@ void MainVisual::createMenu() m_peaksFalloffGroup->addAction(tr("Medium"))->setData(0.2); m_peaksFalloffGroup->addAction(tr("Fast"))->setData(0.4); m_peaksFalloffGroup->addAction(tr("Fastest"))->setData(0.8); - foreach(QAction *act, m_peaksFalloffGroup->actions ()) + for(QAction *act : m_peaksFalloffGroup->actions()) { act->setCheckable(true); peaksFalloff->addAction(act); @@ -302,38 +302,38 @@ void MainVisual::readSettings() { m_update = true; - foreach(QAction *act, m_visModeGroup->actions ()) + for(QAction *act : m_visModeGroup->actions()) { if (vis_name == act->data().toString()) act->setChecked(true); } m_peaksAction->setChecked(settings.value("vis_show_peaks", true).toBool()); int fps = settings.value("vis_rate", 25).toInt(); - foreach(QAction *act, m_fpsGroup->actions ()) + for(QAction *act : m_fpsGroup->actions()) { if (fps == act->data().toInt()) act->setChecked(true); } int mode = settings.value("vis_analyzer_mode", 0).toInt(); - foreach(QAction *act, m_analyzerModeGroup->actions ()) + for(QAction *act : m_analyzerModeGroup->actions ()) { if (mode == act->data().toInt()) act->setChecked(true); } int type = settings.value("vis_analyzer_type", 1).toInt(); - foreach(QAction *act, m_analyzerTypeGroup->actions ()) + for(QAction *act : m_analyzerTypeGroup->actions ()) { if (type == act->data().toInt()) act->setChecked(true); } double speed = settings.value("vis_peaks_falloff", 0.2).toDouble(); - foreach(QAction *act, m_peaksFalloffGroup->actions ()) + for(QAction *act : m_peaksFalloffGroup->actions ()) { if (speed == act->data().toDouble()) act->setChecked(true); } speed = settings.value("vis_analyzer_falloff", 2.2).toDouble(); - foreach(QAction *act, m_analyzerFalloffGroup->actions ()) + for(QAction *act : m_analyzerFalloffGroup->actions ()) { if (speed == act->data().toDouble()) act->setChecked(true); diff --git a/src/plugins/Ui/skinned/mainwindow.cpp b/src/plugins/Ui/skinned/mainwindow.cpp index 14e57f385..e2ca6b9c0 100644 --- a/src/plugins/Ui/skinned/mainwindow.cpp +++ b/src/plugins/Ui/skinned/mainwindow.cpp @@ -279,7 +279,7 @@ void MainWindow::readSettings() QRect availableGeometry = screen->availableGeometry(); QPoint pos = settings.value("mw_pos", QPoint(100, 100)).toPoint(); int r = m_skin->ratio(); - foreach(QScreen *screen, QGuiApplication::screens()) + for(const QScreen *screen : QGuiApplication::screens()) { if(screen->availableGeometry().contains(pos)) { diff --git a/src/plugins/Ui/skinned/playlist.cpp b/src/plugins/Ui/skinned/playlist.cpp index 43f4fc7bf..334fa4bfb 100644 --- a/src/plugins/Ui/skinned/playlist.cpp +++ b/src/plugins/Ui/skinned/playlist.cpp @@ -473,7 +473,7 @@ void PlayList::readSettings() QPoint pos = settings.value ("Skinned/pl_pos", QPoint (100, 332)).toPoint(); m_ratio = m_skin->ratio(); //TODO QGuiApplication::screenAt - foreach(QScreen *screen, QGuiApplication::screens()) + for(const QScreen *screen : QGuiApplication::screens()) { if(screen->availableGeometry().contains(pos)) { @@ -611,7 +611,7 @@ void PlayList::generateCopySelectedMenu() action->setIcon(QIcon::fromTheme("document-new")); m_copySelectedMenu->addSeparator(); - foreach(QString name, m_pl_manager->playListNames()) + for(QString name : m_pl_manager->playListNames()) { m_copySelectedMenu->addAction("&"+name.replace("&", "&&")); } @@ -628,7 +628,7 @@ void PlayList::copySelectedMenuActionTriggered(QAction *action) else { actionText.remove(0,1).replace("&&", "&"); - foreach(PlayListModel *model, m_pl_manager->playLists()) + for(PlayListModel *model : m_pl_manager->playLists()) { //if("&" + model->name().replace("&", "&&") == actionText) if(model->name() == actionText) @@ -644,7 +644,7 @@ void PlayList::copySelectedMenuActionTriggered(QAction *action) return; } QList <PlayListTrack *> theCopy; - foreach(PlayListTrack *track, m_pl_manager->selectedPlayList()->selectedTracks()) + for(PlayListTrack *track : m_pl_manager->selectedPlayList()->selectedTracks()) { PlayListTrack *newItem = new PlayListTrack(*track); theCopy << newItem; diff --git a/src/plugins/Ui/skinned/playlistbrowser.cpp b/src/plugins/Ui/skinned/playlistbrowser.cpp index 56299ffd7..a4db5699f 100644 --- a/src/plugins/Ui/skinned/playlistbrowser.cpp +++ b/src/plugins/Ui/skinned/playlistbrowser.cpp @@ -69,7 +69,7 @@ void PlayListBrowser::updateList() { m_ui.listView->selectionModel()->blockSignals(true); m_listModel->clear(); - foreach(PlayListModel *model, m_pl_manager->playLists()) + for(PlayListModel *model : m_pl_manager->playLists()) { QStandardItem *item = new QStandardItem(model->name()); if(m_pl_manager->currentPlayList() == model) @@ -129,11 +129,11 @@ void PlayListBrowser::rename() void PlayListBrowser::on_deleteButton_clicked() { QModelIndexList selectedIndexes; - foreach(QModelIndex index, m_ui.listView->selectionModel()->selectedIndexes()) + for(const QModelIndex &index : m_ui.listView->selectionModel()->selectedIndexes()) { selectedIndexes << m_proxyModel->mapToSource(index); } - foreach (QModelIndex index, selectedIndexes) + for(const QModelIndex &index : qAsConst(selectedIndexes)) { m_pl_manager->removePlayList(index.row()); } diff --git a/src/plugins/Ui/skinned/playlistheader.cpp b/src/plugins/Ui/skinned/playlistheader.cpp index 0507f2f98..22ee0e023 100644 --- a/src/plugins/Ui/skinned/playlistheader.cpp +++ b/src/plugins/Ui/skinned/playlistheader.cpp @@ -100,7 +100,7 @@ PlayListHeader::PlayListHeader(QWidget *parent) : m_alignmentMenu->addAction(tr("Center", "alignment"))->setData(ListWidgetRow::ALIGN_CENTER); connect(m_alignmentMenu, SIGNAL(triggered(QAction*)), SLOT(setAlignment(QAction*))); QActionGroup *alignmentGroup = new QActionGroup(this); - foreach (QAction *a, m_alignmentMenu->actions()) + for(QAction *a : m_alignmentMenu->actions()) { a->setCheckable(true); alignmentGroup->addAction(a); @@ -276,7 +276,7 @@ int PlayListHeader::maxScrollValue() const return 0; int row_width = 0; - foreach (int size, sizes()) + for(int size : sizes()) { row_width += size; } @@ -637,7 +637,7 @@ void PlayListHeader::contextMenuEvent(QContextMenuEvent *e) m_autoResizeAction->setChecked(m_model->data(m_pressed_column, AUTO_RESIZE).toBool()); int alignment = m_model->data(m_pressed_column, ALIGNMENT).toInt(); - foreach (QAction *action, m_alignmentMenu->actions()) + for(QAction *action : m_alignmentMenu->actions()) { if(action->data().toInt() == alignment) { @@ -647,7 +647,7 @@ void PlayListHeader::contextMenuEvent(QContextMenuEvent *e) } //hide unused actions - foreach (QAction *action, m_menu->actions()) + for(QAction *action : m_menu->actions()) { if(m_menu->actions().at(0) == action) action->setVisible(m_model->count() < MAX_COLUMNS); @@ -660,7 +660,7 @@ void PlayListHeader::contextMenuEvent(QContextMenuEvent *e) } else { - foreach (QAction *action, m_menu->actions()) + for(QAction *action : m_menu->actions()) { if(action == m_menu->actions().first()) action->setVisible(m_model->count() < MAX_COLUMNS); diff --git a/src/plugins/Ui/skinned/playlistselector.cpp b/src/plugins/Ui/skinned/playlistselector.cpp index d3f3006c9..562d95d7f 100644 --- a/src/plugins/Ui/skinned/playlistselector.cpp +++ b/src/plugins/Ui/skinned/playlistselector.cpp @@ -92,7 +92,7 @@ void PlayListSelector::updateTabs() m_rects.clear(); m_extra_rects.clear(); QRect rect; - foreach(QString text, m_pl_manager->playListNames()) + for(const QString &text : m_pl_manager->playListNames()) { if(m_rects.isEmpty()) rect.setX(9); diff --git a/src/plugins/Ui/skinned/preseteditor.cpp b/src/plugins/Ui/skinned/preseteditor.cpp index 87e29f869..7d483601a 100644 --- a/src/plugins/Ui/skinned/preseteditor.cpp +++ b/src/plugins/Ui/skinned/preseteditor.cpp @@ -45,7 +45,7 @@ PresetEditor::~PresetEditor() void PresetEditor::addPresets(const QList<EQPreset*> &presets) { - foreach(QListWidgetItem *item, presets) + for(QListWidgetItem *item : qAsConst(presets)) { ui.presetListWidget->addItem(item); } @@ -53,7 +53,7 @@ void PresetEditor::addPresets(const QList<EQPreset*> &presets) void PresetEditor::addAutoPresets(const QList<EQPreset*> &presets) { - foreach(QListWidgetItem *item, presets) + for(QListWidgetItem *item : qAsConst(presets)) { ui.autoPresetListWidget->addItem(item); } diff --git a/src/plugins/Ui/skinned/shortcutdialog.cpp b/src/plugins/Ui/skinned/shortcutdialog.cpp index 311e53c14..83243c220 100644 --- a/src/plugins/Ui/skinned/shortcutdialog.cpp +++ b/src/plugins/Ui/skinned/shortcutdialog.cpp @@ -28,7 +28,7 @@ ShortcutDialog::ShortcutDialog(const QString &key, QWidget *parent) m_ui.keyLineEdit->setText(key); //buttons should not catch keys - foreach(QAbstractButton *button, m_ui.buttonBox->buttons()) + for(QAbstractButton *button : m_ui.buttonBox->buttons()) button->setFocusPolicy(Qt::NoFocus); } diff --git a/src/plugins/Ui/skinned/skin.cpp b/src/plugins/Ui/skinned/skin.cpp index f379955fb..4b96899c8 100644 --- a/src/plugins/Ui/skinned/skin.cpp +++ b/src/plugins/Ui/skinned/skin.cpp @@ -132,20 +132,19 @@ void Skin::setSkin (const QString& path) loadColors(); if(m_double_size) { - uint key; - foreach(key, m_buttons.keys()) + for(uint key : m_buttons.keys()) m_buttons[key] = scalePixmap(m_buttons[key]); - foreach(key, m_titlebar.keys()) + for(uint key : m_titlebar.keys()) m_titlebar[key] = scalePixmap(m_titlebar[key]); - foreach(key, m_pl_parts.keys()) + for(uint key : m_pl_parts.keys()) m_pl_parts[key] = scalePixmap(m_pl_parts[key]); - foreach(key, m_eq_parts.keys()) + for(uint key : m_eq_parts.keys()) m_eq_parts[key] = scalePixmap(m_eq_parts[key]); - foreach(key, m_ms_parts.keys()) + for(uint key : m_ms_parts.keys()) m_ms_parts[key] = scalePixmap(m_ms_parts[key]); - foreach(key, m_parts.keys()) + for(uint key : m_parts.keys()) m_parts[key] = scalePixmap(m_parts[key]); - foreach(QChar c, m_letters.keys()) + for(const QChar &c : m_letters.keys()) m_letters[c] = scalePixmap(m_letters[c]); m_main = scalePixmap(m_main); posbar = scalePixmap(posbar); @@ -400,7 +399,7 @@ void Skin::loadPlayList() QPixmap *Skin::getPixmap (const QString& name, const QString &fallback) { m_skin_dir.setFilter (QDir::Files | QDir::Hidden | QDir::NoSymLinks); - foreach(QFileInfo info, m_skin_dir.entryInfoList(QStringList() << name + ".*")) + for(const QFileInfo &info : m_skin_dir.entryInfoList(QStringList() << name + ".*")) { if(info.suffix().toLower() != "cur" && info.suffix().toLower() != "txt") return new QPixmap (info.filePath()); @@ -408,7 +407,7 @@ QPixmap *Skin::getPixmap (const QString& name, const QString &fallback) if(!fallback.isEmpty()) { - foreach(QFileInfo info, m_skin_dir.entryInfoList(QStringList() << fallback + ".*")) + for(const QFileInfo &info : m_skin_dir.entryInfoList(QStringList() << fallback + ".*")) { if(info.suffix().toLower() != "cur" && info.suffix().toLower() != "txt") return new QPixmap (info.filePath()); @@ -795,7 +794,7 @@ QRegion Skin::createRegion(const QString &path, const QString &key) QStringList numPoints = settings.value(key+"/NumPoints").toStringList(); QStringList value = settings.value(key+"/PointList").toStringList(); QStringList numbers; - foreach(QString str, value) + for(const QString &str : qAsConst(value)) numbers << str.split(" ", QString::SkipEmptyParts); QList <QRegion> regions; diff --git a/src/plugins/Ui/skinned/skinnedsettings.cpp b/src/plugins/Ui/skinned/skinnedsettings.cpp index 6bc10c20e..141498aa0 100644 --- a/src/plugins/Ui/skinned/skinnedsettings.cpp +++ b/src/plugins/Ui/skinned/skinnedsettings.cpp @@ -119,7 +119,7 @@ void SkinnedSettings::on_skinInstallButton_clicked() { QStringList files = FileDialog::getOpenFileNames(this,tr("Select Skin Files"), QDir::homePath(), tr("Skin files") + " (*.tar.gz *.tgz *.tar.bz2 *.zip *.wsz)"); - foreach(QString path, files) + for(const QString &path : qAsConst(files)) { QFile file(path); file.copy(Qmmp::configDir() + "/skins/" + QFileInfo(path).fileName()); @@ -163,7 +163,7 @@ void SkinnedSettings::findSkins(const QString &path) QList <QFileInfo> fileList = dir.entryInfoList(); if (fileList.count() == 0) return; - foreach (QFileInfo fileInfo, fileList) + for(const QFileInfo &fileInfo : qAsConst(fileList)) { QPixmap preview = Skin::getPixmap ("main", QDir(fileInfo.filePath ())); if (!preview.isNull()) @@ -203,7 +203,7 @@ void SkinnedSettings::loadSkins() #else findSkins(Qmmp::dataPath()); #endif - foreach(QString path, m_reader->skins()) + for(const QString &path : m_reader->skins()) { QListWidgetItem *item = new QListWidgetItem (path.section('/', -1)); item->setIcon (m_reader->getPreview(path)); diff --git a/src/plugins/Ui/skinned/skinreader.cpp b/src/plugins/Ui/skinned/skinreader.cpp index 0494af7ea..ae3faa305 100644 --- a/src/plugins/Ui/skinned/skinreader.cpp +++ b/src/plugins/Ui/skinned/skinreader.cpp @@ -60,10 +60,10 @@ void SkinReader::generateThumbs() cache_dir.setFilter(QDir::Files | QDir::Hidden | QDir::NoSymLinks); QFileInfoList d = cache_dir.entryInfoList(); //clear removed skins from cache - foreach(QFileInfo thumbFile, d) + for(const QFileInfo &thumbFile : qAsConst(d)) { bool del = true; - foreach(QFileInfo fileInfo, f) + for(const QFileInfo &fileInfo : qAsConst(f)) { if (fileInfo.baseName () == thumbFile.baseName ()) { @@ -80,10 +80,10 @@ void SkinReader::generateThumbs() } } //add new skins to cache - foreach(QFileInfo fileInfo, f) + for(const QFileInfo &fileInfo : qAsConst(f)) { bool create = true; - foreach(QFileInfo thumbInfo, d) + for(const QFileInfo &thumbInfo : qAsConst(d)) { if (fileInfo.baseName () == thumbInfo.baseName ()) { @@ -106,9 +106,9 @@ void SkinReader::generateThumbs() //add thumbs to map cache_dir.refresh(); d = cache_dir.entryInfoList(); - foreach(QFileInfo fileInfo, f) + for(const QFileInfo &fileInfo : qAsConst(f)) { - foreach(QFileInfo thumbInfo, d) + for(const QFileInfo &thumbInfo : qAsConst(d)) { if (fileInfo.baseName () == thumbInfo.baseName ()) { @@ -126,7 +126,7 @@ void SkinReader::unpackSkin(const QString &path) QDir dir(Qmmp::configDir() + "/cache/skin"); dir.setFilter( QDir::Files | QDir::Hidden | QDir::NoSymLinks); QFileInfoList f = dir.entryInfoList(); - foreach(QFileInfo file, f) + for(const QFileInfo &file : qAsConst(f)) dir.remove(file.fileName()); //unpack QString name = QFileInfo(path).fileName().toLower(); @@ -157,7 +157,7 @@ void SkinReader::untar(const QString &from, const QString &to, bool preview) array = m_process->readAllStandardOutput (); QString str = QString(array); QStringList outputList = str.split("\n", QString::SkipEmptyParts); - foreach(QString str, outputList) + for(QString str : qAsConst(outputList)) { str = str.trimmed(); args.clear(); @@ -194,7 +194,7 @@ void SkinReader::unzip(const QString &from, const QString &to, bool preview) QDir dir(to); dir.setFilter(QDir::Files | QDir::Hidden | QDir::NoSymLinks); QFileInfoList fileList = dir.entryInfoList(); - foreach(QFileInfo thumbInfo, fileList) + for(const QFileInfo &thumbInfo : qAsConst(fileList)) { if (thumbInfo.fileName().startsWith("main.", Qt::CaseInsensitive)) { diff --git a/src/plugins/Ui/skinned/visualmenu.cpp b/src/plugins/Ui/skinned/visualmenu.cpp index cf99d361a..6135eda10 100644 --- a/src/plugins/Ui/skinned/visualmenu.cpp +++ b/src/plugins/Ui/skinned/visualmenu.cpp @@ -25,8 +25,7 @@ VisualMenu::VisualMenu(QWidget *parent) : QMenu(tr("Visualization"), parent) { - VisualFactory *factory = nullptr; - foreach(factory, Visual::factories()) + for(VisualFactory *factory : Visual::factories()) { VisualAction *act = new VisualAction(factory, this); addAction(act); diff --git a/src/plugins/Visual/analyzer/analyzer.cpp b/src/plugins/Visual/analyzer/analyzer.cpp index 2b10f84fe..b0a40c44f 100644 --- a/src/plugins/Visual/analyzer/analyzer.cpp +++ b/src/plugins/Visual/analyzer/analyzer.cpp @@ -117,17 +117,17 @@ void Analyzer::readSettings() m_update = true; m_peaksAction->setChecked(m_show_peaks); - foreach(QAction *act, m_fpsGroup->actions ()) + for(QAction *act : m_fpsGroup->actions ()) { if (m_timer->interval() == 1000 / act->data().toInt()) act->setChecked(true); } - foreach(QAction *act, m_peaksFalloffGroup->actions ()) + for(QAction *act : m_peaksFalloffGroup->actions ()) { if (m_peaks_falloff == act->data().toDouble()) act->setChecked(true); } - foreach(QAction *act, m_analyzerFalloffGroup->actions ()) + for(QAction *act : m_analyzerFalloffGroup->actions ()) { if (m_analyzer_falloff == act->data().toDouble()) act->setChecked(true); @@ -339,7 +339,7 @@ void Analyzer::createMenu() m_fpsGroup->addAction(tr("25 fps"))->setData(25); m_fpsGroup->addAction(tr("10 fps"))->setData(10); m_fpsGroup->addAction(tr("5 fps"))->setData(5); - foreach(QAction *act, m_fpsGroup->actions ()) + for(QAction *act : m_fpsGroup->actions ()) { act->setCheckable(true); refreshRate->addAction(act); @@ -353,7 +353,7 @@ void Analyzer::createMenu() m_analyzerFalloffGroup->addAction(tr("Medium"))->setData(2.2); m_analyzerFalloffGroup->addAction(tr("Fast"))->setData(2.4); m_analyzerFalloffGroup->addAction(tr("Fastest"))->setData(2.8); - foreach(QAction *act, m_analyzerFalloffGroup->actions ()) + for(QAction *act : m_analyzerFalloffGroup->actions()) { act->setCheckable(true); analyzerFalloff->addAction(act); @@ -367,7 +367,7 @@ void Analyzer::createMenu() m_peaksFalloffGroup->addAction(tr("Medium"))->setData(0.2); m_peaksFalloffGroup->addAction(tr("Fast"))->setData(0.4); m_peaksFalloffGroup->addAction(tr("Fastest"))->setData(0.8); - foreach(QAction *act, m_peaksFalloffGroup->actions ()) + for(QAction *act : m_peaksFalloffGroup->actions()) { act->setCheckable(true); peaksFalloff->addAction(act); diff --git a/src/plugins/Visual/projectm/projectmwidget.cpp b/src/plugins/Visual/projectm/projectmwidget.cpp index 1bc4d75c2..163846cf0 100644 --- a/src/plugins/Visual/projectm/projectmwidget.cpp +++ b/src/plugins/Visual/projectm/projectmwidget.cpp @@ -112,7 +112,7 @@ void ProjectMWidget::initializeGL() RatingList list; list.push_back(3); list.push_back(3); - foreach (QFileInfo info, l) + for(const QFileInfo &info : qAsConst(l)) { m_projectM->addPresetURL (info.absoluteFilePath().toStdString(), info.fileName().toStdString(), list); m_listWidget->addItem(info.fileName()); |