[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