>               {
> -                     if (((GList *) node->data)->data == tag)
> -                             break;
> +                     GList *node;
> +                     /* should always be the first element as we returned 
> the first one in
> +                      * tags_table_lookup() */
> +                     foreach_list(node, list)
> +                     {
> +                             if (((GList *) node->data)->data == tag)
> +                                     break;
> +                     }
> +                     list = g_list_delete_link(list, node);
> +                     g_tree_insert(tree, GINT_TO_POINTER(tag->line), list);

shouldn't the entry be removed if `list==NULL` like it used to do?

---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/797/files#r54357906

Reply via email to