[brisbane-core MERGE] CHKInventory.iter_non_root_entries()

[Ian Clatworthy]

Ian Clatworthy wrote:

> The current user of the API - loading texts during a commit - doesn't
> want it so it does simplify the calling code not returning it. I don't
> really mind either way though - I was mostly reimplementing the entries()
> API without the path bit. My original cut at this gave entries() an
> include-path=True parameter but that didn't turn out because the client
> code wanted an iterator, not list.
> 
> I'm happy to go with whatever API name and behaviour you recommend.

We can leave this on hold - my latest code doesn't need it at all.
It may still be useful though to keep in mind as calculating paths that
aren't needed is pretty common I suspect.

Right now in particular, path calculation is ultra slow. I'll take
a look at that.

Ian C.