a.sidorin added inline comments.

================
Comment at: lib/StaticAnalyzer/Checkers/VforkChecker.cpp:47
@@ +46,3 @@
+
+  bool isChildProcess(const ProgramStateRef State) const;
+
----------------
ygribov wrote:
> a.sidorin wrote:
> > I think it's a good idea to make some functions static and/or move them out 
> > of class definition.
> Right. Which one is preferred btw?
isChildProcess(), getAssignedVariable() are good candidates since they don't 
use any class member.


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