NoQ added inline comments.

================
Comment at: cfe/trunk/lib/StaticAnalyzer/Checkers/CStringChecker.cpp:295-297
+      OS << "Null pointer argument in call to " << CurrentFunctionDescription
+         << ' ' << IdxOfArg << llvm::getOrdinalSuffix(IdxOfArg)
+         << " parameter";
----------------
It sounds like this code remained untested because tests don't match 
end-of-line.

I noticed it because it produces warnings that don't look like valid English, 
such as:

> "Null pointer argument in call to memory set function 1st parameter"

Patches are welcome :)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D66333



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D66333: [analyzer]... Artem Dergachev via Phabricator via cfe-commits

Reply via email to