[PATCH] D76605: [analyzer] Display the checker name in the text output

Kristóf Umann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 7 07:01:06 PDT 2020


Szelethus added inline comments.


================
Comment at: clang/lib/StaticAnalyzer/Core/TextDiagnostics.cpp:112
         reportPiece(NoteID, Piece->getLocation().asLocation(),
-                    Piece->getString(), Piece->getRanges(), Piece->getFixits());
+                    Piece->getString().str(), Piece->getRanges(),
+                    Piece->getFixits());
----------------
Szelethus wrote:
> NoQ wrote:
> > Szelethus wrote:
> > > martong wrote:
> > > > Why the `.str()` ?
> > > `StringRef` no longer converts to `std::string` implicitly.
> > But it seems to have been fine before(?)
> Mind that I changed the parameter of `reportPiece` to `std::string` from `StringRef`.
Couldn't see the forest behind the trees :^)


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

https://reviews.llvm.org/D76605





More information about the cfe-commits mailing list