Elijah Newren <new...@gmail.com> writes: > == The patch == > - Ben's patch only affects the "checkout -b $NEWBRANCH" case. He > checks for it by looking for any other flag that would be a different > case, and using the old codepath if he finds any. > - This means there is a "giant list of checks" for this optimization, > and an ongoing maintenance burden because if anyone ever adds any > extra options, this optimization might suddenly break things if that > giant list of checks isn't updated.
Correct. Having said that, I do not mind leaving it in 'pu' for the rest of the cycle, possibly merging it to 'next' after that and keeping it there, to see how horrible a maintenance burden it would become in real life, though.
