dcoughlin added a comment. Other than one teeny nit, looks good to me.
================ Comment at: lib/StaticAnalyzer/Checkers/CStringChecker.cpp:863 @@ +862,3 @@ + + const ElementRegion *ER = dyn_cast<ElementRegion>(R); + // Cast is safe as BufVal's region is a FieldRegion. ---------------- You can use cast<ElementRegion>(R) here, which will assert that R is an ElementRegion, and remove the assert below. http://reviews.llvm.org/D11832 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
