Andres Freund <and...@anarazel.de> writes: > On 2023-01-22 18:20:49 +0100, Jelte Fennema wrote: >> I don't think the amount of pain is really much lower if we reformat >> 10,000 or 300,000 lines of code, without automation both would be >> quite painful. But the git commands I shared in my previous email >> should alleviate most of that pain.
> It's practically not possible to review a 300k line change. And perhaps I'm > paranoid, but I would have a problem with a commit in the history that's > practically not reviewable. As far as that goes, if you had concern then you could run the indentation tool locally and confirm you got matching results. But this does point up that the processes Jelte suggested all depend critically on indentation results being 100% reproducible by anybody. So the more I think about it the less excited I am about depending on clang-format, because version skew in peoples' clang installations seems inevitable, and there's good reason to fear that that would show up as varying indentation results. regards, tom lane