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

bugzilla-daemon at libre-soc.org bugzilla-daemon at libre-soc.org
Mon Jun 12 22:05:05 BST 2023


--- Comment #135 from Dmitry Selyutin <ghostmansd at gmail.com> ---
What I like here:
1. There's no need to update classes with some magic methods. If the walker's
handler is registered, this will be caught.
2. It's loosely typed; inheritance is taken into account.
3. This has no problems with builtins and their "walkability".
4. The use is dead-simple.

What I don't like:
1. I don't know if it's possible to make super() work with it (that is, call
the superclass handler). Not sure if we need it, though: common things can be
moved to usual methods or functions, and these can be called in hooks.
2. Perhaps this is more complex than it should've been. I tried to make it
generic, though, and somewhat flexible.

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

More information about the libre-soc-bugs mailing list