#263 mfs findnext code looks wrond

[Stas Sergeev]

I am probably misreading the code, but
I can't see how the hlist stack works.
sdb_p_cluster(sdb), which is supposed
to carry an index in stack, is only
updated after hlist_push(). But I don't
see it being updated after hlist_pop()
or hlist_pop_psp(), so I wonder why is
there a stack: it looks like only the
last entry is being processed, and there
is no way to the prev entry, ever.
Also, I wonder what this code does:

---

  if (se->psp == psp)
    se->duplicates++;
  free_list(se, TRUE);
  goto exit;

---

Shouldn't it look as follows:

  if (se->psp == psp) {
    se->duplicates++;
    free_list(se, TRUE);
    goto exit;
  }

---