@kugel- requested changes on this pull request.
Almost. In principle I approve the change in the current state.
> @@ -628,34 +631,44 @@ static void save_ui_prefs(GKeyFile *config)
}
}
+typedef enum ConfigPayload
+{
+ PREFERENCES,
Minor nitpick: Can you please shorten this to PREFS (and above PREFERENCES_FILE
-> PREFS_FILE)? We usually refer to the configuration as "prefs" throughout the
source and this would follow the pattern.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2776#pullrequestreview-628847906
