Commit 1312b772 authored by Michael Pyne's avatar Michael Pyne

Finish removing deprecated code.

Aside from -Wsuggest-override errors in auto-generated Qt and Taglib
code, JuK compiles without warning again.
parent 3b703e1b
Pipeline #3625 passed with stage
in 7 minutes and 9 seconds
......@@ -40,6 +40,7 @@
#include <QTime>
#include <QTimer>
#include <QDesktopWidget>
#include <QScreen>
#include <QStatusBar>
#include <QDBusMessage>
#include <QDBusReply>
......@@ -121,7 +122,8 @@ JuK::JuK(const QStringList &filesToOpen, QWidget *parent) :
if(firstRun) {
QRect r = rect();
r.moveCenter(QApplication::desktop()->screenGeometry().center());
const QRect screenCenter = QApplication::primaryScreen()->availableGeometry();
r.moveCenter(screenCenter.center());
move(r.topLeft());
}
......
......@@ -44,6 +44,7 @@
#include <QKeyEvent>
#include <QHBoxLayout>
#include <QVBoxLayout>
#include <QRegularExpression>
#include <QSizePolicy>
#include <id3v1genres.h>
......@@ -495,7 +496,8 @@ void TagEditor::setupLayout()
// Do some meta-programming to find the matching enable boxes
for(auto enable : findChildren<QCheckBox *>(QRegExp("Enable$"))) {
const auto enableCheckBoxes = findChildren<QCheckBox *>(QRegularExpression("Enable$"));
for(auto enable : enableCheckBoxes) {
enable->hide(); // These are shown only when multiple items are being edited
// Each enable checkbox is identified by having its objectName end in "Enable".
......@@ -514,7 +516,8 @@ void TagEditor::setupLayout()
// Make sure that the labels are as tall as the enable boxes so that the
// layout doesn't jump around as the enable boxes are shown/hidden.
for(auto label : findChildren<QLabel *>()) {
const auto editorLabels = findChildren<QLabel *>();
for(auto label : editorLabels) {
if(m_enableBoxes.contains(label->buddy()))
label->setMinimumHeight(m_enableBoxes[label->buddy()]->height());
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment