[PATCH] D44557: [analyzer] CStringChecker.cpp - Code refactoring on bug report.
Henry Wong via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 16 05:29:15 PDT 2018
MTC created this revision.
MTC added reviewers: NoQ, george.karpenkov, xazax.hun.
Herald added subscribers: cfe-commits, a.sidorin, rnkovacs, szepet.
When improving the modeling `evalMemset()`, some scenes need to emit report of `NotNullTerm`. In this case,
there are three places in `CStringChecker.cpp` that have the same code about `NotNullTerm` report. In
addition, `CStringChecker.cpp` already has `emitOverlapBug()`, so I think it needs code refactoring.
Thanks in advance for the code review!
Repository:
rC Clang
https://reviews.llvm.org/D44557
Files:
lib/StaticAnalyzer/Checkers/CStringChecker.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44557.138682.patch
Type: text/x-patch
Size: 11125 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180316/7c09580a/attachment.bin>
More information about the cfe-commits
mailing list