Commit d0659544 authored by Laurent Montel's avatar Laurent Montel 😁
Browse files

Fix clazy warning

parent 0a9572ca
Pipeline #23183 passed with stage
in 31 minutes and 35 seconds
......@@ -88,7 +88,7 @@ void SendLaterManager::createSendInfoList()
//Look at QQueue
if (mSendLaterQueue.isEmpty()) {
if (!mListSendLaterInfo.isEmpty()) {
mCurrentInfo = mListSendLaterInfo.first();
mCurrentInfo = mListSendLaterInfo.constFirst();
const QDateTime now = QDateTime::currentDateTime();
const qint64 seconds = now.secsTo(mCurrentInfo->dateTime());
if (seconds > 0) {
......
......@@ -131,7 +131,8 @@ void SecurityPage::ReadingTab::save()
KStandardGuiItem::cont(), KStandardGuiItem::cancel(), QStringLiteral("htmlMailOverride")) == KMessageBox::Continue) {
saveCheckBox(mSGTab.mHtmlMailCheck, MessageViewer::MessageViewerSettings::self()->htmlMailItem());
if (kmkernel) {
for (const Akonadi::Collection &collection : kmkernel->allFolders()) {
const auto allFolders = kmkernel->allFolders();
for (const Akonadi::Collection &collection : allFolders) {
KConfigGroup config(KMKernel::self()->config(), MailCommon::FolderSettings::configGroupName(collection));
//Old config
config.deleteEntry("htmlMailOverride");
......
......@@ -2245,6 +2245,7 @@ bool KMComposerWin::insertFromMimeData(const QMimeData *source, bool forceAttach
if (items.isEmpty() && collections.isEmpty()) {
if (allLocalURLs || forceAttachment) {
QList<AttachmentInfo> infoList;
infoList.reserve(urlList.count());
for (const QUrl &url : urlList) {
AttachmentInfo info;
info.url = url;
......
......@@ -59,7 +59,7 @@ void RemoveDuplicateMailJob::start()
Akonadi::RemoveDuplicatesJob *job = new Akonadi::RemoveDuplicatesJob(collections, this);
job->setProperty("ProgressItem", QVariant::fromValue(item));
item->setProperty("RemoveDuplicatesJob", QVariant::fromValue(qobject_cast<Akonadi::Job *>(job)));
item->setProperty("RemoveDuplicatesJob", QVariant::fromValue(job));
connect(job, &KJob::finished, this, &RemoveDuplicateMailJob::slotRemoveDuplicatesDone);
connect(job, &KJob::description, this, &RemoveDuplicateMailJob::slotRemoveDuplicatesUpdate);
connect(item, &KPIM::ProgressItem::progressItemCanceled, this, &RemoveDuplicateMailJob::slotRemoveDuplicatesCanceled);
......
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