[PATCH] D24915: [analyzer] Extend bug reports with extra notes - ObjCDeallocChecker

2016-10-04 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL283093: [analyzer] Add extra notes to ObjCDeallocChecker (authored by dergachev). Changed prior to commit: https://reviews.llvm.org/D24915?vs=72520&id=73407#toc Repository: rL LLVM https://reviews.l

[PATCH] D24915: [analyzer] Extend bug reports with extra notes - ObjCDeallocChecker

2016-10-03 Thread Vitaly Buka via cfe-commits
vitalybuka added a comment. Reverted with r283181 as it depends r283092 https://reviews.llvm.org/D24915 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D24915: [analyzer] Extend bug reports with extra notes - ObjCDeallocChecker

2016-09-27 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. LGTM and Devin's comments have been addressed. https://reviews.llvm.org/D24915 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http

Re: [PATCH] D24915: [analyzer] Extend bug reports with extra notes - ObjCDeallocChecker

2016-09-26 Thread Artem Dergachev via cfe-commits
NoQ updated this revision to Diff 72520. NoQ added a comment. Addressed two inline comments by Devin in the original review https://reviews.llvm.org/D24278. Rebased on top of the updated https://reviews.llvm.org/D24278. https://reviews.llvm.org/D24915 Files: lib/StaticAnalyzer/Checkers/Chec

[PATCH] D24915: [analyzer] Extend bug reports with extra notes - ObjCDeallocChecker

2016-09-26 Thread Artem Dergachev via cfe-commits
NoQ created this revision. NoQ added reviewers: zaks.anna, dcoughlin. NoQ added a subscriber: cfe-commits. This patch applies the bug reporting technique that is being introduced in D24278 to ObjCDeallocChecker. Extra notes highlight instance variable or property declarations that are of relevan