Enforce static registry to be a function, not a variable.

Author: wengxt

src/CMakeLists.txt

@@ -14,7 +14,7 @@ if (ENABLE_SERVER)
         fcitx5_get_addon_targets(ADDON_TARGETS modules frontend im ui)
 
         fcitx5_import_addons(fcitx5
-            REGISTRY_VARNAME staticAddon
+            REGISTRY_VARNAME getStaticAddon
             ADDONS ${ADDON_TARGETS}
         )
     endif()

src/lib/fcitx-utils/Fcitx5Macros.cmake

@@ -24,7 +24,7 @@ function(fcitx5_import_addons target)
       set(filename "${CMAKE_CURRENT_BINARY_DIR}/${target}-${addon}-import-addon.cpp")
       file(WRITE "${filename}" "
 #include <fcitx/addonloader.h>
-extern fcitx::StaticAddonRegistry ${FCITX5_IMPORT_REGISTRY_VARNAME};
+extern fcitx::StaticAddonRegistry ${FCITX5_IMPORT_REGISTRY_VARNAME}();
 FCITX_IMPORT_ADDON_FACTORY(${FCITX5_IMPORT_REGISTRY_VARNAME}, ${addon});
 ")
       target_sources(${target} PRIVATE ${filename})

src/lib/fcitx/addoninstance.h

@@ -208,20 +208,26 @@ class FCITXCORE_EXPORT AddonInstance {
     }                                                                          \
     }
 
+#define FCITX_DEFINE_STATIC_ADDON_REGISTRY(Name, ...)                          \
+    ::fcitx::StaticAddonRegistry &Name() {                                     \
+        static ::fcitx::StaticAddonRegistry registry{__VA_ARGS__};             \
+        return registry;                                                       \
+    }
+
 #define FCITX_ADDON_FACTORY_V2_BACKWARDS(AddonName, ClassName)                 \
     FCITX_ADDON_FACTORY_V2(AddonName, ClassName)                               \
     FCITX_ADDON_FACTORY(ClassName)
 
-#define FCITX_IMPORT_ADDON_FACTORY(StaticRegistry, AddonName)                  \
+#define FCITX_IMPORT_ADDON_FACTORY(StaticRegistryGetter, AddonName)            \
     extern "C" {                                                               \
     ::fcitx::AddonFactory *fcitx_addon_factory_instance_##AddonName();         \
     }                                                                          \
     class StaticAddonRegistrar_##AddonName {                                   \
     public:                                                                    \
         StaticAddonRegistrar_##AddonName() {                                   \
-            (StaticRegistry)                                                   \
-                .emplace(FCITX_STRINGIFY(AddonName),                           \
-                         fcitx_addon_factory_instance_##AddonName());          \
+            (StaticRegistryGetter)().emplace(                                  \
+                FCITX_STRINGIFY(AddonName),                                    \
+                fcitx_addon_factory_instance_##AddonName());                   \
         }                                                                      \
     };                                                                         \
     StaticAddonRegistrar_##AddonName staticAddonRegistrar_##AddonName

src/server/main.cpp

@@ -31,9 +31,9 @@ using namespace fcitx;
 int selfpipe[2];
 std::string crashlog;
 
-StaticAddonRegistry staticAddon;
+FCITX_DEFINE_STATIC_ADDON_REGISTRY(getStaticAddon)
 #ifdef ENABLE_KEYBOARD
-FCITX_IMPORT_ADDON_FACTORY(staticAddon, keyboard);
+FCITX_IMPORT_ADDON_FACTORY(getStaticAddon, keyboard);
 #endif
 
 int main(int argc, char *argv[]) {
@@ -71,7 +71,7 @@ int main(int argc, char *argv[]) {
         Instance instance(argc, argv);
         instance.setBinaryMode();
         instance.setSignalPipe(selfpipe[0]);
-        instance.addonManager().registerDefaultLoader(&staticAddon);
+        instance.addonManager().registerDefaultLoader(&getStaticAddon());
 
         ret = instance.exec();
         restart = instance.isRestartRequested();

test/testcompose.cpp

@@ -22,7 +22,7 @@
 
 using namespace fcitx;
 
-StaticAddonRegistry staticAddon;
+FCITX_DEFINE_STATIC_ADDON_REGISTRY(staticAddon);
 FCITX_IMPORT_ADDON_FACTORY(staticAddon, keyboard);
 
 void scheduleEvent(EventDispatcher *dispatcher, Instance *instance) {
@@ -96,7 +96,7 @@ int main() {
                 instance.privateData()->xkbContext_.get(), testCompose,
                 FCITX_ARRAY_SIZE(testCompose), "", XKB_COMPOSE_FORMAT_TEXT_V1,
                 XKB_COMPOSE_COMPILE_NO_FLAGS));
-        instance.addonManager().registerDefaultLoader(&staticAddon);
+        instance.addonManager().registerDefaultLoader(&staticAddon());
         EventDispatcher dispatcher;
         dispatcher.attach(&instance.eventLoop());
         scheduleEvent(&dispatcher, &instance);

test/testspell.cpp

@@ -21,7 +21,7 @@
 
 using namespace fcitx;
 
-StaticAddonRegistry staticAddon;
+FCITX_DEFINE_STATIC_ADDON_REGISTRY(staticAddon);
 FCITX_IMPORT_ADDON_FACTORY(staticAddon, keyboard);
 
 void scheduleEvent(EventDispatcher *dispatcher, Instance *instance) {
@@ -78,7 +78,7 @@ int main() {
     char arg2[] = "--enable=keyboard,testfrontend,spell,testui";
     char *argv[] = {arg0, arg1, arg2};
     Instance instance(FCITX_ARRAY_SIZE(argv), argv);
-    instance.addonManager().registerDefaultLoader(&staticAddon);
+    instance.addonManager().registerDefaultLoader(&staticAddon());
     EventDispatcher dispatcher;
     dispatcher.attach(&instance.eventLoop());
     scheduleEvent(&dispatcher, &instance);