The problem is not that the time waiting on a code review can't be used in some other way. It's that the constant interruptions seriously hurt straight-line productivity. "OK, done with this minor patch. Now, uhhh, while I wait for code review, I guess I'll go review this other thing... And then start a new patch.... Code review is done now? Better context-switch back."
I'm not strictly opposed to pre-commit reviews, but I think it's incorrect to be dismissive of their cost.
I'm not strictly opposed to pre-commit reviews, but I think it's incorrect to be dismissive of their cost.