Compare commits

...

2 Commits

Author SHA1 Message Date
Berthold Stoeger
5a751f0eee preferences: replace macro by local function
No apparent reason for having this as a hard-to-read macro.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
2024-03-09 11:11:09 +01:00
Berthold Stoeger
d6c515fd85 preferences: replace SIGNAL/SLOT by function pointers
This give compile time checking. In fact, one of the connections was
not working (currentIndexChanged(QString) doesn't exist in newer(?)
Qt versions).

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
2024-03-09 11:11:09 +01:00
6 changed files with 12 additions and 13 deletions

View File

@ -96,7 +96,7 @@ void PreferencesCloud::syncSettings()
return; return;
} }
CloudStorageAuthenticate *cloudAuth = new CloudStorageAuthenticate(this); CloudStorageAuthenticate *cloudAuth = new CloudStorageAuthenticate(this);
connect(cloudAuth, SIGNAL(finishedAuthenticate()), this, SLOT(updateCloudAuthenticationState())); connect(cloudAuth, &CloudStorageAuthenticate::finishedAuthenticate, this, &PreferencesCloud::updateCloudAuthenticationState);
cloudAuth->backend(email, password, pin); cloudAuth->backend(email, password, pin);
} }
} }

View File

@ -37,4 +37,3 @@ void PreferencesDc::refreshSettings()
void PreferencesDc::syncSettings() void PreferencesDc::syncSettings()
{ {
} }

View File

@ -77,14 +77,19 @@ void PreferencesGraph::syncSettings()
qPrefTechnicalDetails::set_display_deco_mode(ui->vpmb->isChecked() ? VPMB : BUEHLMANN); qPrefTechnicalDetails::set_display_deco_mode(ui->vpmb->isChecked() ? VPMB : BUEHLMANN);
} }
#define DANGER_GF (gf > 100) ? "* { color: red; }" : "" static const char *danger_gf(int gf)
{
return (gf > 100) ? "* { color: red; }" : "";
}
void PreferencesGraph::on_gflow_valueChanged(int gf) void PreferencesGraph::on_gflow_valueChanged(int gf)
{ {
ui->gflow->setStyleSheet(DANGER_GF); ui->gflow->setStyleSheet(danger_gf(gf));
} }
void PreferencesGraph::on_gfhigh_valueChanged(int gf) void PreferencesGraph::on_gfhigh_valueChanged(int gf)
{ {
ui->gfhigh->setStyleSheet(DANGER_GF); ui->gfhigh->setStyleSheet(danger_gf(gf));
} }
void PreferencesGraph::on_buehlmann_toggled(bool buehlmann) void PreferencesGraph::on_buehlmann_toggled(bool buehlmann)
@ -96,5 +101,3 @@ void PreferencesGraph::on_buehlmann_toggled(bool buehlmann)
ui->vpmb_conservatism->setEnabled(!buehlmann); ui->vpmb_conservatism->setEnabled(!buehlmann);
ui->label_VPMB->setEnabled(!buehlmann); ui->label_VPMB->setEnabled(!buehlmann);
} }
#undef DANGER_GF

View File

@ -30,8 +30,8 @@ PreferencesLanguage::PreferencesLanguage() : AbstractPreferencesWidget(tr("Langu
foreach (QString format, dateFormatShortMap.keys()) foreach (QString format, dateFormatShortMap.keys())
ui->dateFormatEntry->addItem(format); ui->dateFormatEntry->addItem(format);
ui->dateFormatEntry->completer()->setCaseSensitivity(Qt::CaseSensitive); ui->dateFormatEntry->completer()->setCaseSensitivity(Qt::CaseSensitive);
connect(ui->dateFormatEntry, SIGNAL(currentIndexChanged(const QString&)), connect(ui->dateFormatEntry, &QComboBox::currentTextChanged,
this, SLOT(dateFormatChanged(const QString&))); this, &PreferencesLanguage::dateFormatChanged);
ui->timeFormatEntry->addItem("hh:mm"); ui->timeFormatEntry->addItem("hh:mm");
ui->timeFormatEntry->addItem("h:mm AP"); ui->timeFormatEntry->addItem("h:mm AP");

View File

@ -17,7 +17,7 @@ PreferencesNetwork::PreferencesNetwork() : AbstractPreferencesWidget(tr("Network
ui->proxyType->addItem(tr("SOCKS proxy"), QNetworkProxy::Socks5Proxy); ui->proxyType->addItem(tr("SOCKS proxy"), QNetworkProxy::Socks5Proxy);
ui->proxyType->setCurrentIndex(-1); ui->proxyType->setCurrentIndex(-1);
connect(ui->proxyType, SIGNAL(currentIndexChanged(int)), this, SLOT(proxyType_changed(int))); connect(ui->proxyType, QOverload<int>::of(&QComboBox::currentIndexChanged), this, &PreferencesNetwork::proxyType_changed);
} }
PreferencesNetwork::~PreferencesNetwork() PreferencesNetwork::~PreferencesNetwork()
@ -47,7 +47,6 @@ void PreferencesNetwork::syncSettings()
proxy->set_proxy_pass(ui->proxyPassword->text()); proxy->set_proxy_pass(ui->proxyPassword->text());
} }
void PreferencesNetwork::proxyType_changed(int idx) void PreferencesNetwork::proxyType_changed(int idx)
{ {
if (idx == -1) { if (idx == -1) {
@ -63,4 +62,3 @@ void PreferencesNetwork::proxyType_changed(int idx)
ui->proxyPassword->setEnabled(hpEnabled & ui->proxyAuthRequired->isChecked()); ui->proxyPassword->setEnabled(hpEnabled & ui->proxyAuthRequired->isChecked());
ui->proxyAuthRequired->setChecked(ui->proxyAuthRequired->isChecked()); ui->proxyAuthRequired->setChecked(ui->proxyAuthRequired->isChecked());
} }

View File

@ -30,4 +30,3 @@ void PreferencesReset::refreshSettings()
void PreferencesReset::syncSettings() void PreferencesReset::syncSettings()
{ {
} }