NoQ added inline comments.

================
Comment at: clang/lib/StaticAnalyzer/Checkers/ReturnValueChecker.cpp:86
+  std::string Name = "";
+  Name += dyn_cast<CXXMethodDecl>(Call.getDecl())->getParent()->getName();
+  Name += "::";
----------------
Either `cast<>` or check for null.


================
Comment at: clang/lib/StaticAnalyzer/Checkers/ReturnValueChecker.cpp:116
+
+        BR.markInteresting(SFC);
+
----------------
This isn't the stack frame i was talking about, but if you move this code to 
`checkEndFunction` it will be.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63915/new/

https://reviews.llvm.org/D63915



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

Reply via email to