Bug report: NewStaticOutline

My company's just purchased a "with source" update to our old iSEDQuickPDF library, and I'm currently going through our old (antique!) code and checking everything builds and works with the new library linked in.

In the course of this I've found what I'm 99.9% sure is a glitch in the NewStaticOutline method; in v7.19 the meat of this function is:

  if ValidOutlineID(Parent) then
  begin
    Result := [Stuff - removed to preserve copyright];
  end else
    Result := 0;

...which fails (returns 0) if a Parent value of 0 (= add root, rather than child, StaticOutline element) is specified as ValidOutlineID doesn't regard 0/Nil as a valid memory location.

The fix for this is - I think - to add the additional condition to the if statement so it becomes:

  if (Parent = 0) or ValidOutlineID(Parent) then

That line, with the additional condition, is already in place in the very similar NewOutline method.

Cheers,
Ian

P.S. Hope this is the right place for a bug report - couldn't see anywhere specific for them on the web page or in the forums.

Done, Rowan - will do the same for the documentation omission I just raised too!