Hello Robert,
> I'm open to having better support for multiple callbacks, I'm not at
> point in dev work where its easy for me to dive in to what can be a
> complicated topic, nor review submissiosns. Once things free up a bit
> I get the backlong of submissions merged I'll add a few more comment
Hi J-S,
I'm open to having better support for multiple callbacks, I'm not at
point in dev work where its easy for me to dive in to what can be a
complicated topic, nor review submissiosns. Once things free up a bit
I get the backlong of submissions merged I'll add a few more comments
on the topic
Hello Robert,
> All present NodeCallbacks are effectively traversal callbacks - all
> require you to call traverse.
Yes, but most NodeCallbacks will be standalone (i.e. no dependency
between callbacks), so calling them in series shouldn't be a problem,
or am I missing something?
Anyways, the
Hi J-S,
All present NodeCallbacks are effectively traversal callbacks - all
require you to call traverse.
On thing I've considered in the past is to have a CompositeCallback
and a callback type within this that passes an custom callback
iterator that either calls the next callback in the chain or
Hello Robert,
> You can nested callbacks within each other right now. It's a bit
> awkward but does work for doing multiple callbacks.
Yes, as I said I find the current nested callback mechanism less
intuitive. It would be better if it were transparent to users of the
class, which is what I
HI J-S,
You can nested callbacks within each other right now. It's a bit
awkward but does work for doing multiple callbacks.
The issued you have with your approach is that you call the callbacks
in series and don't provide a mechanism for traversal - rather it
assumes that each callback would ca
Hello Robert,
I was wondering about something. I find the nesting of node callbacks
a bit less intuitive and transparent than it could be, and I was
wondering if there would be interest in a small class I made called
CompositeNodeCallback. It's simply a vector of NodeCallbacks where the
c
7 matches
Mail list logo