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

Fix clazy warning

parent 7987d9ab
......@@ -25,6 +25,7 @@
class MailDir20PercentAsRead : public MailDir
{
Q_OBJECT
public:
MailDir20PercentAsRead();
void runTest() override;
......
......@@ -25,6 +25,7 @@
class MailDirFetchAllHeaders : public MailDir
{
Q_OBJECT
public:
MailDirFetchAllHeaders();
void runTest() override;
......
......@@ -25,6 +25,7 @@
class MailDirFetchUnreadHeaders : public MailDir
{
Q_OBJECT
public:
MailDirFetchUnreadHeaders();
void runTest() override;
......
......@@ -25,6 +25,7 @@
class MailDirRemoveReadMessages : public MailDir
{
Q_OBJECT
public:
MailDirRemoveReadMessages();
void runTest() override;
......
......@@ -24,6 +24,7 @@
class VCard : public MakeTest
{
Q_OBJECT
public:
VCard(const QString &dir);
VCard();
......
......@@ -39,10 +39,10 @@ public:
};
Q_SIGNALS:
void result(Result);
void result(Akonadi::CommandBase::Result);
protected Q_SLOTS:
virtual void emitResult(Result result);
virtual void emitResult(Akonadi::CommandBase::Result result);
};
}
#endif // COMMANDBASE_H
......@@ -38,7 +38,7 @@ public:
void execute() override;
protected Q_SLOTS:
void emitResult(Result result) override;
void emitResult(Akonadi::CommandBase::Result result) override;
private:
void expunge(const Akonadi::Collection &col);
......
......@@ -50,7 +50,7 @@ public:
bool mRecursive;
};
MarkAsCommand::MarkAsCommand(const Akonadi::MessageStatus &targetStatus, const Akonadi::Item::List &msgList, bool invert, QObject *parent)
MarkAsCommand::MarkAsCommand(Akonadi::MessageStatus targetStatus, const Akonadi::Item::List &msgList, bool invert, QObject *parent)
: CommandBase(parent)
, d(new Akonadi::MarkAsCommandPrivate())
{
......@@ -61,7 +61,7 @@ MarkAsCommand::MarkAsCommand(const Akonadi::MessageStatus &targetStatus, const A
d->mMarkJobCount = 0;
}
MarkAsCommand::MarkAsCommand(const Akonadi::MessageStatus &targetStatus, const Akonadi::Collection::List &folders, bool invert, bool recursive, QObject *parent)
MarkAsCommand::MarkAsCommand(Akonadi::MessageStatus targetStatus, const Akonadi::Collection::List &folders, bool invert, bool recursive, QObject *parent)
: CommandBase(parent)
, d(new Akonadi::MarkAsCommandPrivate())
{
......
......@@ -35,8 +35,8 @@ class AKONADI_MIME_EXPORT MarkAsCommand : public CommandBase
{
Q_OBJECT
public:
MarkAsCommand(const Akonadi::MessageStatus &targetStatus, const Akonadi::Item::List &msgList, bool invert = false, QObject *parent = nullptr);
MarkAsCommand(const Akonadi::MessageStatus &targetStatus, const Akonadi::Collection::List &folders, bool invert = false, bool recursive = false, QObject *parent = nullptr);
MarkAsCommand(MessageStatus targetStatus, const Akonadi::Item::List &msgList, bool invert = false, QObject *parent = nullptr);
MarkAsCommand(MessageStatus targetStatus, const Akonadi::Collection::List &folders, bool invert = false, bool recursive = false, QObject *parent = nullptr);
~MarkAsCommand() override;
void execute() override;
......
......@@ -59,17 +59,17 @@ Akonadi::MessageStatus::MessageStatus()
mStatus = StatusUnknown;
}
bool Akonadi::MessageStatus::operator==(const Akonadi::MessageStatus &other) const
bool Akonadi::MessageStatus::operator==(Akonadi::MessageStatus other) const
{
return mStatus == other.mStatus;
}
bool Akonadi::MessageStatus::operator!=(const Akonadi::MessageStatus &other) const
bool Akonadi::MessageStatus::operator!=(Akonadi::MessageStatus other) const
{
return mStatus != other.mStatus;
}
bool Akonadi::MessageStatus::operator&(const Akonadi::MessageStatus &other) const
bool Akonadi::MessageStatus::operator&(Akonadi::MessageStatus other) const
{
if (mStatus == StatusUnread) {
return !(other.mStatus & StatusRead);
......@@ -87,7 +87,7 @@ void Akonadi::MessageStatus::clear()
mStatus = StatusUnknown;
}
void Akonadi::MessageStatus::set(const Akonadi::MessageStatus &other)
void Akonadi::MessageStatus::set(Akonadi::MessageStatus other)
{
Q_ASSERT(!(other.mStatus & StatusUnread));
......@@ -148,7 +148,7 @@ void Akonadi::MessageStatus::set(const Akonadi::MessageStatus &other)
}
}
void Akonadi::MessageStatus::toggle(const Akonadi::MessageStatus &other)
void Akonadi::MessageStatus::toggle(Akonadi::MessageStatus other)
{
Q_ASSERT(!(other.mStatus & StatusUnread));
......
......@@ -57,20 +57,20 @@ public:
@return true if the stati are equal, false if different.
@param other message status to compare with current object
*/
bool operator==(const MessageStatus &other) const;
bool operator==(MessageStatus other) const;
/** Compare the status with that from another instance.
@return true if the stati are equal, false if different.
@param other message status to compare with current object
*/
bool operator!=(const MessageStatus &other) const;
bool operator!=(MessageStatus other) const;
/** Check, if some of the flags in the status match
with those flags from another instance.
@return true if at least one flag is set in both stati.
@param other message status to compare objects' flags
*/
bool operator&(const MessageStatus &other) const;
bool operator&(MessageStatus other) const;
/** Clear all status flags, this resets to unknown. */
void clear();
......@@ -82,14 +82,14 @@ public:
integrity of the resulting status.
@param other message status to set
*/
void set(const MessageStatus &other);
void set(MessageStatus other);
/** Toggle one or more stati described by another MessageStatus object.
Internally the setters are used to ensure the integrity of the
resulting status.
@param other message status to toggle
*/
void toggle(const MessageStatus &other);
void toggle(MessageStatus other);
/* ----- getters ----------------------------------------------------- */
......
......@@ -39,7 +39,7 @@ public:
private Q_SLOTS:
void slotFetchDone(KJob *job);
void slotMoveDone(Result result);
void slotMoveDone(Akonadi::CommandBase::Result result);
private:
void moveMessages();
......
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