Szelethus added a comment. Herald added a subscriber: gamesh411. In https://reviews.llvm.org/D54438#1297602, @NoQ wrote:
> Hmm, makes sense. Maybe `static bool BlahBlahChecker::isApplicable(const > LangOpts &LO)` or something like that. > > Btw, maybe instead of default constructor and `->setup(Args)` method we could > stuff all of the `setup(Args)`'s arguments into the one-and-only constructor > for the checker? Neat. Make all checkers have only a single constructor, expecting a `const CheckerManager &`! Since `getCurrentName` and the like will be thing of the past, for once, maybe we could also initialize checker options in constructors too. Hopefully. Repository: rC Clang https://reviews.llvm.org/D54438 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
