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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to