diff --git a/src/modules/quickphrase/quickphrase.cpp b/src/modules/quickphrase/quickphrase.cpp
index b499a4b9..01291c50 100644
--- a/src/modules/quickphrase/quickphrase.cpp
+++ b/src/modules/quickphrase/quickphrase.cpp
@@ -10,6 +10,7 @@
 #include "fcitx-config/iniparser.h"
 #include "fcitx-utils/i18n.h"
 #include "fcitx-utils/inputbuffer.h"
+#include "fcitx/addonfactory.h"
 #include "fcitx/addonmanager.h"
 #include "fcitx/candidatelist.h"
 #include "fcitx/inputcontextmanager.h"
diff --git a/src/modules/quickphrase/quickphrase.h b/src/modules/quickphrase/quickphrase.h
index 8dc41985..8b3f2412 100644
--- a/src/modules/quickphrase/quickphrase.h
+++ b/src/modules/quickphrase/quickphrase.h
@@ -12,7 +12,6 @@
 #include "fcitx-config/iniparser.h"
 #include "fcitx-utils/i18n.h"
 #include "fcitx-utils/key.h"
-#include "fcitx/addonfactory.h"
 #include "fcitx/addoninstance.h"
 #include "fcitx/inputcontextproperty.h"
 #include "fcitx/instance.h"
@@ -59,7 +58,7 @@ class QuickPhrase final : public AddonInstance {
         safeSaveAsIni(config_, "conf/quickphrase.conf");
     }
     void setSubConfig(const std::string &path,
-                      const fcitx::RawConfig &) override {
+                      const fcitx::RawConfig & /*unused*/) override {
         if (path == "editor") {
             reloadConfig();
         }