@b4n requested changes on this pull request.

Apart from the small nitpick from my compiler, looks good and seems to indeed 
work nicely (tested that it e.g. fixed *tests/ctags/oop.sv*, and couldn't find 
any regression) :+1: 

> @@ -324,6 +324,56 @@ static gint compare_symbol_lines(gconstpointer a, 
> gconstpointer b)
 }
 
 
+static gboolean is_symbol_within_parent(const TMTag *tag, const TMTag *parent)
+{
+       const gchar *scope_sep = tm_parser_scope_separator(tag->lang);
+       guint scope_len = 0;

```suggestion
        gsize scope_len = 0;
```

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/4063#pullrequestreview-2829537870
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/4063/review/[email protected]>

Reply via email to