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