codebrainz commented on this pull request.
@@ -736,13 +739,9 @@ static gboolean on_button_press(GtkWidget *widget, GdkEventButton *event, gpoint
} else if (event->button == 3) { - static GtkWidget *popup_menu = NULL;
While I dislike static local (or global) variables which allocate memory dynamically and never cleanup, moving this one to the global scope seems completely unrelated to this change. Or did I miss it being used elsewhere?