Jeff King <peff@xxxxxxxx> writes: > I'd have expected it with individual callbacks like: > > int handle_patience(...) > { > do_patience_specific_stuff(); > do_shared_stuff(PATIENCE_DIFF); > } > > int handle_histogram(...) > { > do_shared_stuff(HISTOGRAM_DIFF); > } > > and so on. That's a bit more verbose, but the call stack reflects the > flow we expect. Exactly. Thanks for spelling it out.