On Tue, May 14, 2019 at 12:49:17PM -0700, Julius Werner wrote:
One thing I'm missing is an option to maintain clang-format as an optional pre-submit hook that people can choose run over only the lines their patch is touching if they want to, like I suggested in that thread above.
Since everybody is free to do that without project coordination, we don't need to ask for that to be project policy, no?
Well, we could still maintain a default .clang-format file and prepare a ready-to-install commit hook so that people don't have to figure out the details of clang-format-diff theirselves. I think it presents a valid third way of "how do we want to deal with auto-formatting".
I don't think there's anything bad enough to warrant calling off the current round though, which would only create additional confusion.
Yes, I didn't want to suggest that. I guess people who agree that formatting should be optional can vote against it here and then we could discuss details later if the option wins.