@techee commented on this pull request.


>               {
                        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 or view it on GitHub:
https://github.com/geany/geany/pull/3269#discussion_r974431564
You are receiving this because you are subscribed to this thread.

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

Reply via email to