@techee commented on this pull request.


In src/tagmanager/tm_workspace.c:

> @@ -746,9 +847,12 @@ static gint sort_found_tags(gconstpointer a, gconstpointer b, gpointer user_data
 	const TMTag *t1 = *((TMTag **) a);
 	const TMTag *t2 = *((TMTag **) b);
 
-	/* sort local vars first (with highest line number first), followed
-	 * by tags from current file, followed by workspace tags, followed by
-	 * global tags */
+	/* sort local vars first (with highest line number first),

IMO this is not something that should be documented. For users this whole thing is "geany does its best to place more relevant symbols to the top of the list" and how Geany does this is an implementation detail. You won't really notice this logic in practice, just that the symbol you want to type (hopefully) appears at or near the top of the list. If we happen to add support for LSP, each server will have its own logic which will be totally different from what we use in Geany now.


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <geany/geany/pull/3269/review/1089914771@github.com>