codebrainz commented on this pull request.
@@ -2440,20 +2441,32 @@ void ui_init_builder(void)
gtk_builder_set_translation_domain(builder, GETTEXT_PACKAGE);
error = NULL; - interface_file = g_build_filename(app->datadir, "geany.glade", NULL); - if (! gtk_builder_add_from_file(builder, interface_file, &error)) + ui_data = g_resource_lookup_data(geany_get_resource(), + "/org/geany/Geany/geany.glade", G_RESOURCE_LOOKUP_FLAGS_NONE, &error);
Too bad we can't use gtk_builder_add_from_resource()… maybe we should simply add compatibility for it on GTK < 3.4?
Yeah. Alternatively we could just add a comment there saying to change the code when the needed version is minimum supported.
I think this specific (example) code is missing an `#else`, and also I think the `gtk_build_add_from_string` call is not OK because of the undefined ordering of parameter evaluation in C (which is why I did it with two calls in the patch).