@techee commented on this pull request.


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

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

Reply via email to