@techee commented on this pull request.


In src/tagmanager/tm_workspace.c:

>  		{
 			guint j;
 
-			for (j = 0; j < tm_files->len; j++)
+			if (!*header_candidates)

So I guess the idea is to avoid multiple assignments to *header_candidates in case a file is included multiple times (maybe legitimately, via different directories)?

This is just an optimization so the code inside the if block doesn't have to be executed if we already found header_candidates.


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/1113792547@github.com>