@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)

The check

if (!*header_candidates)

is to avoid doing the comparisons below if we already set header_candidates in the previous iteration. Also notice *header_candidates = NULL; at the beginning of the function.

What you describe would be

if (!header_candidates)

(i.e. without the dereference)


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