Typz added a comment. In D50078#2050623 <https://reviews.llvm.org/D50078#2050623>, @MyDeveloperDay wrote:
> I get the point about Opt In, and if we are going to add an option it needs > to go in ASAP otherwise too many people will then start complaining it was on > by default before. > > To be honest I think the question is do we consider this a bug or a feature, > because we DO NOT provide Opt In on bug fixes (and there is no Opt Out). > > If its a bug, there is no need for us to not have this on by default, its > better to drive the improvement than 10,000's of .clang-format files needing > to be updated just to get reasonable behavior. I certainly a bit biased, but I think it is more a bug: it should have been there from the start :-) And I also think it should be on by default instead of modifying many .clang-format files. So IMHO if we add an option, it should be opt-out. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D50078/new/ https://reviews.llvm.org/D50078 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
