[PATCH] D48291: [analyzer][UninitializedObjectChecker] Fixed captured lambda variable name

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 6 08:58:46 PDT 2018

Szelethus marked 3 inline comments as done.
Szelethus added inline comments.

Comment at: lib/StaticAnalyzer/Checkers/UninitializedObjectChecker.cpp:691
+StringRef getVariableName(const FieldDecl *Field) {
+  // If \p Field is a captured lambda variable, Field->getName() will return
george.karpenkov wrote:
> Is this a member method? Then it should be prefixed with a class name.
> Currently it looks like you have a member method without an implementation and a separate C-style function?
It is a statically defined method, and you can see its forward declaration in the same diff.

Back when I started writing this checker, the amount of function laying in the anonymous namespace was very few, but this has changed. I'll fix it sometime because it's getting somewhat annoying (and is against the [[ https://llvm.org/docs/CodingStandards.html | coding standards ]]).


More information about the cfe-commits mailing list