[PATCH] D77062: [analyzer] Improve zero assumption in CStringChecke::assumeZero

Denys Petrov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 4 06:12:17 PDT 2020


ASDenysPetrov added a comment.
Herald added a subscriber: steakhal.

Ping!



================
Comment at: clang/lib/StaticAnalyzer/Checkers/CStringChecker.cpp:199
   // Utility methods
-  std::pair<ProgramStateRef , ProgramStateRef >
-  static assumeZero(CheckerContext &C,
-                    ProgramStateRef state, SVal V, QualType Ty);
+  std::pair<ProgramStateRef, ProgramStateRef> static assumeZero(
+      ProgramStateRef state, SVal V);
----------------
vsavchenko wrote:
> Can you please put `static` before return type, it will be more consistent with other static methods nearby.
Sure, I just didn't pay attention to that.


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

https://reviews.llvm.org/D77062



More information about the cfe-commits mailing list