[Github-comments] [geany/geany] Start porting to GSettings (#1257)

elextr notifications at xxxxx
Sat Oct 8 02:03:45 UTC 2016


elextr commented on this pull request.



> @@ -41,6 +41,14 @@ static void on_sidebar_pos_left_changed(GSettings *settings, gchar *key, gpointe
 }
 
 
+static void on_editor_font_changed(GSettings *settings, gchar *key, gpointer user_data)
+{

I was hoping that there was enough similarity between them to maybe have a few callbacks and pass sufficient information via the `user_data`.  For example this type of setting would handle all string settings that call a set function, and the user data would need a pointer to the setting of course, and a pointer to a function to call (set to `ui_set_editor_font()` for this setting).

-- 
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/1257
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20161007/578ba40f/attachment.html>


More information about the Github-comments mailing list