Jonathan Tan <jonathantanmy@xxxxxxxxxx> writes: >> Sounds good. Jonathan? I've squashed Josh'es Reviewed-by, but I >> will refrain from merging it to 'next' just yet to see if you too >> like the proposed code structure. > > I think that this is a local enough concern that going either way won't > paint us into a corner, so if what's in > jt/connectivity-check-optim-in-partial-clone is OK, I prefer using that > to reduce churn. If you do not think their improvement is not much of improvement, then please say so. On the other hand, if you also believe that the resulting code is better, adopting the improvement (in other words, help from other people) and get a better version of the code queued before it hits 'next' is not a "churn". Leaving a chance to make the code into a shape that you think better is a waste and risking the chance to forget improving it with a follow-up patch. Thanks.
