From b01652f54d784352141d69a8e9e70ba6995662d9 Mon Sep 17 00:00:00 2001 From: Simon Redman Date: Fri, 26 Oct 2018 15:29:28 -0600 Subject: [PATCH] Fix include guards Summary: Some plugins were copy/pasted from a template and forgot to change their include guards. Since plugin header files are never cross-referenced, this was not a serious issue, but it looks nicer to be correct NotificationListener.h did not have an include guard. As before, this is not a problem currently, but it's best to have it fixed Test Plan: Project should build, compile, and run as normal Reviewers: #kde_connect, nicolasfella Reviewed By: #kde_connect, nicolasfella Subscribers: kdeconnect Tags: #kde_connect Differential Revision: https://phabricator.kde.org/D16437 --- plugins/findmyphone/findmyphoneplugin.h | 6 +++--- plugins/runcommand/runcommand_config.h | 6 +++--- plugins/sendnotifications/notificationslistener.h | 5 +++++ 3 files changed, 11 insertions(+), 6 deletions(-) diff --git a/plugins/findmyphone/findmyphoneplugin.h b/plugins/findmyphone/findmyphoneplugin.h index eee1f80bb..4c8e90f38 100644 --- a/plugins/findmyphone/findmyphoneplugin.h +++ b/plugins/findmyphone/findmyphoneplugin.h @@ -18,8 +18,8 @@ * along with this program. If not, see . */ -#ifndef PINGPLUGIN_H -#define PINGPLUGIN_H +#ifndef FINDMYPHONEPLUGIN_H +#define FINDMYPHONEPLUGIN_H #include @@ -44,4 +44,4 @@ public: bool receivePacket(const NetworkPacket& np) override; }; -#endif +#endif // FINDMYPHONEPLUGIN_H diff --git a/plugins/runcommand/runcommand_config.h b/plugins/runcommand/runcommand_config.h index 7efe7b7b6..02d24090b 100644 --- a/plugins/runcommand/runcommand_config.h +++ b/plugins/runcommand/runcommand_config.h @@ -18,8 +18,8 @@ * along with this program. If not, see . */ -#ifndef SHARE_CONFIG_H -#define SHARE_CONFIG_H +#ifndef RUNCOMMAND_CONFIG_H +#define RUNCOMMAND_CONFIG_H #include "kcmplugin/kdeconnectpluginkcm.h" @@ -50,4 +50,4 @@ private: }; -#endif +#endif // RUNCOMMAND_CONFIG_H diff --git a/plugins/sendnotifications/notificationslistener.h b/plugins/sendnotifications/notificationslistener.h index 48b875a38..bcb821dc0 100644 --- a/plugins/sendnotifications/notificationslistener.h +++ b/plugins/sendnotifications/notificationslistener.h @@ -18,6 +18,9 @@ * along with this program. If not, see . */ +#ifndef NOTIFICATIONLISTENER_H +#define NOTIFICATIONLISTENER_H + #include #include #include @@ -62,3 +65,5 @@ private: void setTranslatedAppName(); QString m_translatedAppName; }; + +#endif // NOTIFICATIONLISTENER_H