[PATCH] D14014: Checker of proper vfork usage
Yury Gribov via cfe-commits
cfe-commits at lists.llvm.org
Sat Oct 24 08:20:55 PDT 2015
ygribov added inline comments.
================
Comment at: lib/StaticAnalyzer/Checkers/VforkChecker.cpp:149
@@ +148,3 @@
+
+ // see if it's an ordinary assignment
+ do {
----------------
ygribov wrote:
> a.sidorin wrote:
> > You can use early return to escape do{}.
> In this particular case - yes. But what's wrong about original do-while-false? I thought it's a common idiom...
FYI there are several uses of do-while-false in CSA codebase.
Repository:
rL LLVM
http://reviews.llvm.org/D14014
More information about the cfe-commits
mailing list