@kugel- commented on this pull request.


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

You are right, I didn't notice the outer loop. 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)?

So I think this is alright now.

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

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

Reply via email to