[Libre-soc-bugs] [Bug 1094] insndb instruction database visitor-walker is needed

bugzilla-daemon at libre-soc.org bugzilla-daemon at libre-soc.org
Wed Jun 21 22:26:48 BST 2023


https://bugs.libre-soc.org/show_bug.cgi?id=1094

--- Comment #224 from Luke Kenneth Casson Leighton <lkcl at lkcl.net> ---
(In reply to Dmitry Selyutin from comment #223)

> Already done.

awesome.

> Well, handling paths is a real-world usage. We have `pysvp64db tree`
> command, which needs exactly that. Yes this _is_ taking so long, because I
> want to cover cases I have in mind,

can you do it as a sub-class or mix-in class?  not part of the primary
API but an "example that someone might commonly use"?  and if it becomes
common enough it gets moved *into* mdis?

> You're mixing two different unrelated things. 

almost certainly. i'm not able to keep up fully at the moment, sorry.

> The object itself and the way
> we reached it. Anyway, as for naming, I think "node" is a perfect name. Are
> you fine with it?

yes.

> This is not about who yields the path. This is about what's yielded. You're
> probably speaking of something else. My question is whether the path needs
> to be full (to root, upon which we started walking) or to relative (parent).

relative.

> Anyway, I'm mostly fine with how it looks. I'll return to paths issue when I
> handle `pysvp64db tree` command.

subclass or mixin.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


More information about the libre-soc-bugs mailing list