<div dir="ltr">Hi all,<div><br></div><div>I&#39;m wondering if enforcing clang-format for all patches is a good idea...</div><div><br></div><div>I&#39;ve recently seen patches where clang-format is doing changes on parts of the code that have not been touched by the patch. Given that all files were already formatted by clang-format long ago, this shouldn&#39;t happen.</div><div><br></div><div>This means that as the clang-format version evolves, the formatting with the same configuration is not the same. This introduces unnecessary noise to the file history that I think it should be avoided.</div><div><br></div><div>Additionally, I&#39;ve also seen some cases where some constructs are reformatted in an uglier or less clear way. I think it&#39;s very hard to come up with a set of rules that formats everything in the best possible way.</div><div><br></div><div>For all these reasons, I would say we shouldn&#39;t enforce clang-format to accept a PR. I think it&#39;s a good test to run to catch some clear formatting issues, but it shouldn&#39;t vote for patch acceptance.</div><div><br></div><div>What do you think ?</div><div><br></div><div>Regards,</div><div><br></div><div>Xavi</div></div>