@eht16 commented on this pull request.
> GtkTreeViewColumn *column;
kbdata->tree = GTK_TREE_VIEW(ui_lookup_widget(ui_widgets.prefs_dialog,
"treeview7"));
kbdata->store = gtk_tree_store_new(KB_TREE_COLUMNS,
G_TYPE_STRING, G_TYPE_STRING, G_TYPE_INT, G_TYPE_BOOLEAN,
G_TYPE_INT);
- gtk_tree_view_set_model(GTK_TREE_VIEW(kbdata->tree),
GTK_TREE_MODEL(kbdata->store));
+ filter_model = gtk_tree_model_filter_new(GTK_TREE_MODEL(kbdata->store),
NULL);
+ gtk_tree_model_filter_set_visible_func(
+ GTK_TREE_MODEL_FILTER(filter_model), kb_tree_filter_func,
kb_filter_entry, NULL);
+ /* set model to tree view */
+ gtk_tree_view_set_model(GTK_TREE_VIEW(kbdata->tree), filter_model);
+ g_object_unref(filter_model);
Is it correct to unref both objects?
It seems to work this way but I'm never sure with unref'ing.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/4192#pullrequestreview-2530864653
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany/pull/4192/review/[email protected]>