On 2021-11-04 06:59, Jamal Hadi Salim wrote:

On 2021-11-03 13:12, Joe Burton wrote:

quoted

That's a good point. Since the probe is invoked before the update takes
place, it would not be possible to account for the possibility that the
update failed.

Unless someone wants the `pre update' hook, I'll simply adjust the
existing hooks' semantics so that they are invoked after the update.
As discussed, this better suits the intended use case.

If the goal is to synchronize state between two maps (if i understood
correctly the intent) then it is more useful to go post-update.

To complete that thought: Only positive results are interesting.
For example if the command was to delete an entry which doesnt
exist there is no point in reporting that (or is there?).
OTOH, a successful delete is useful...

cheers,
jamal