Szelethus accepted this revision. Szelethus added a comment. In D65182#1630540 <https://reviews.llvm.org/D65182#1630540>, @NoQ wrote:
> In D65182#1629192 <https://reviews.llvm.org/D65182#1629192>, @Szelethus wrote: > > > Hmm, why the need for checker options? Why not have them by default? If > > fixits are an experimental feature, maybe we should have a global > > `enable-fixits` config. But I don't insist :) > > > I did this because fixits have a much higher quality standard that warnings > (because people often apply them blindly). This means that fixit support is > likely to be lagging behind in meeting the quality standards and therefore > it's important to allow some (but not necessarily all) on-by-default checkers > have alpha fixits. I'm sold. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65182/new/ https://reviews.llvm.org/D65182 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
