[PATCH] D12852: [Static Analyzer] Moving nullability checkers to a top level package.

2015-09-14 Thread Gábor Horváth via cfe-commits
xazax.hun created this revision. xazax.hun added reviewers: zaks.anna, dcoughlin, jordan_rose. xazax.hun added a subscriber: cfe-commits. This patch is a preparation to enabling some of the nullability checks by default. 1. Moved nullability checkers to a top level package. The core package is

Re: [PATCH] D12852: [Static Analyzer] Moving nullability checkers to a top level package.

2015-09-14 Thread Gábor Horváth via cfe-commits
xazax.hun updated this revision to Diff 34705. xazax.hun added a comment. Removed an unintended comment formatting change. http://reviews.llvm.org/D12852 Files: lib/StaticAnalyzer/Checkers/Checkers.td lib/StaticAnalyzer/Checkers/NullabilityChecker.cpp test/Analysis/nullability.mm

Re: [PATCH] D12852: [Static Analyzer] Moving nullability checkers to a top level package.

2015-09-14 Thread Anna Zaks via cfe-commits
zaks.anna accepted this revision. zaks.anna added a comment. This revision is now accepted and ready to land. Please, explain in code why you are doing this. Otherwise, LGTM. Comment at: lib/StaticAnalyzer/Checkers/NullabilityChecker.cpp:139 @@ -138,2 +138,3 @@

Re: [PATCH] D12852: [Static Analyzer] Moving nullability checkers to a top level package.

2015-09-14 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL247590: [Static Analyzer] Moving nullability checkers to a top level package. (authored by xazax). Changed prior to commit: http://reviews.llvm.org/D12852?vs=34705=34713#toc Repository: rL LLVM