Szelethus marked an inline comment as done. Szelethus added inline comments.
================ Comment at: lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp:527-530 + // developer checkers even in the alpha output. For example, + // alpha.cplusplus.IteratorModeling is a modeling checker, hence it's hidden + // by default, and users (even when the user is a developer of an alpha + // checker) shouldn't normally tinker with whether they should be enabled. ---------------- Szelethus wrote: > NoQ wrote: > > Hmm, just thought about this: by this logic, should we hide all `core` > > checkers? Because we don't really support users tinkering with whether they > > should be enabled :) > Thats actually a very cool idea. We should totally do that! Well umm. In a sense, sure, but wouldn't we want to display the checker descriptions at least? Let's not rush this for now. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62093/new/ https://reviews.llvm.org/D62093 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
