Charusso marked an inline comment as done. Charusso added inline comments.
================ Comment at: clang/lib/StaticAnalyzer/Core/BugReporterVisitors.cpp:855 + + CurrentSFC = Node->getStackFrame(); + ---------------- NoQ wrote: > Mmm, wait a sec. This way the loop condition is trivially true. The second > check on line 862 is also trivially true. I think you want to make it a > separate variable. I was never cool in algorithms, thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62926/new/ https://reviews.llvm.org/D62926 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
