[PATCH] D79358: [analyzer] CERT: STR37-C

2020-05-05 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a comment. I'm afraid so. The patch otherwise looked really clean, sorry to ruin the day! :) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79358/new/ https://reviews.llvm.org/D79358 __

[PATCH] D79358: [analyzer] CERT: STR37-C

2020-05-05 Thread Zurab Tsinadze via Phabricator via cfe-commits
zukatsinadze added a comment. In D79358#2019755 , @Szelethus wrote: > Adding @martong, because I fear that this is colliding with > StdLibraryFunctionsChecker. The warnings added here seem to be, in essence, > identical to D73898

[PATCH] D79358: [analyzer] CERT: STR37-C

2020-05-05 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a reviewer: martong. Szelethus added a comment. Herald added a subscriber: rnkovacs. Adding @martong, because I fear that this is colliding with StdLibraryFunctionsChecker. The warnings added here seem to be, in essence, identical to D73898 . Re

[PATCH] D79358: [analyzer] CERT: STR37-C

2020-05-04 Thread Zurab Tsinadze via Phabricator via cfe-commits
zukatsinadze created this revision. zukatsinadze added reviewers: NoQ, Charusso, Szelethus. zukatsinadze added a project: clang. Herald added subscribers: cfe-commits, ASDenysPetrov, martong, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun, mgorny. This patc