"Matt Hunter" [off-list ref] writes:
I should explain myself better. I don't mean Colin should avoid
splitting out logic from the main flow in do_fetch. But the end result
can be reached without introducing a new function. I find needing to
hold fewer functions in my head makes the code easier to reason about,
though that is a subjective point.
The idea in my head is below and is
Signed-off-by: Matt Hunter <redacted>
Feel free to run with this or not.
Hmph, interesting. Can you rewrite Colin's patch using this, and
then merge the result into 'seen' (after reverting Colin's topic
ch/remote-defer-follow-remote-head-validation from 'seen' first) to
see how badly it crashes with other topics in flight?
Thanks.