@codebrainz I prefer my implementation for the compat function, but either is fine. Maybe your checks for emptiness are nice though (as add_from_string()
doesn't accept it for some reason, although from_resource()
does), although it really shouldn't ever happen.
Otherwise, I think the changes in ui_utils.c
are slightly better in mine, and closer to master (less unnecessary changes):
diff --git a/src/ui_utils.c b/src/ui_utils.c
index a0093174a..5b544bff4 100644
--- a/src/ui_utils.c
+++ b/src/ui_utils.c
@@ -42,7 +42,6 @@
#include "msgwindow.h"
#include "prefs.h"
#include "project.h"
-#include "resources.h"
#include "sciwrappers.h"
#include "sidebar.h"
#include "stash.h"
@@ -2427,7 +2426,6 @@ static GtkWidget *ui_get_top_parent(GtkWidget *widget)
void ui_init_builder(void)
{
const gchar *name;
- GBytes *ui_data;
GError *error;
GSList *iter, *all_objects;
GtkWidget *widget, *toplevel;
@@ -2445,7 +2443,7 @@ void ui_init_builder(void)
{
dialogs_show_msgbox_with_secondary(GTK_MESSAGE_ERROR,
_("Geany cannot start!"), error->message);
- g_error(_("Cannot load user-interface: %s"), error->message);
+ g_error("Cannot create user-interface: %s", error->message);
g_error_free(error);
g_object_unref(builder);
return;
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.