[PATCH] D52787: [analyzer][NFC] Refactor functions in PlistDiagnostics to take AnalyzerOptions as parameter

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 2 10:50:23 PDT 2018


Szelethus added inline comments.


================
Comment at: lib/StaticAnalyzer/Core/PlistDiagnostics.cpp:306-307
+                       const FIDMap& FM,
+                       const Preprocessor &PP,
+                       AnalyzerOptions &AnOpts) {
+  ReportPiece(o, P, FM, PP, AnOpts, /*indent*/ 4, /*depth*/ 0,
----------------
NoQ wrote:
> Maybe just make this function non-`static` instead of passing members around manually?
That would probably be super-invasive for this file, from what I'm seeing. I might end up doing that, but this would mean pretty much a complete refactoring.

To be honest, almost all functions here should be a method, the output stream, `FIDMap`, `Preprocessor`, `AnalyzerOptions` could all be left out.


Repository:
  rC Clang

https://reviews.llvm.org/D52787





More information about the cfe-commits mailing list