[PATCH] D30341: [analyzer] clarify error messages about uninitialized function arguments

[Daniel Marjamäki via Phabricator via cfe-commits]

danielmarjamaki marked an inline comment as done.
danielmarjamaki added inline comments.


================
Comment at: lib/StaticAnalyzer/Checkers/CallAndMessageChecker.cpp:211
       // Generate a report for this bug.
-      StringRef Desc =
-          describeUninitializedArgumentInCall(Call, IsFirstArgument);
+      std::string Desc =
+          describeUninitializedArgumentInCall(Call, ArgumentNumber);
----------------
zaks.anna wrote:
> Have you considered using  llvm::raw_svector_ostream here as well as passing it an argument to describeUninitializedArgumentInCall? For example, see  MallocChecker.cpp.
I changed so describeUninitializedArgumentInCall() returns an llvm::Twine instead of std::string. hope you like it.



Repository:
  rL LLVM

https://reviews.llvm.org/D30341