On Mon, Sep 29, 2014 at 3:35 AM, Henrik Grubbström (Lysator) @ Pike (-) developers forum 10353@lyskom.lysator.liu.se wrote:
Program.all_inherits has a FIXME asking for docs. But its definition is a little odd: it returns a list of the given program's inherits, and everything they inherit - but not recursively.
Not quite... Take a closer look at the loop -- It isn't a foreach()...
I could document it as such, but it strikes me that this should be recursive, similar to inherit_tree but returning a flat array. Example implementation in branch rosuav/pgm_all_inherits - what do you think?
No need for a new implementation; the current already does it.
Oohhhhh...... right. I looked at it and misread it as a manually-implemented foreach. Okay. In that case, I'll document it as-is, but also add an explanatory comment for the benefit of the next person.
ChrisA