[PATCH] D14303: [analyzer] Add 'optin' checker package and move localizability checkers into it.

[Devin Coughlin via cfe-commits]

dcoughlin created this revision.
dcoughlin added reviewers: zaks.anna, krememek.
dcoughlin added a subscriber: cfe-commits.

This patch creates a new 'optin' top-level checker package and moves several of the localizability checkers into it. 

This package is for checkers that are not alpha and that would normally be on by default but where the driver does not have enough information to determine when they are applicable. The localizability checkers fit this criterion because the driver cannot determine whether a project is localized or not -- this is best determined at the IDE or build-system level.

This new package is *not* intended for checkers that are too noisy to be on by default.

The hierarchy under 'optin' mirrors that in 'alpha': checkers under 'optin' should be organized in the hierarchy they would have had if they were truly top level (e.g., optin.osx.cocoa.MyOptInChecker).

We are not attached to the name 'optin'. If you have ideas for a better name, we welcome suggestions.

http://reviews.llvm.org/D14303

Files:
  lib/StaticAnalyzer/Checkers/Checkers.td
  lib/StaticAnalyzer/Checkers/LocalizationChecker.cpp
  test/Analysis/localization-aggressive.m
  test/Analysis/localization.m

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D14303.39102.patch
Type: text/x-patch
Size: 5067 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20151103/183bcf91/attachment.bin>