[PATCH] D58367: [analyzer] NFC: Improve upon the concept of BugReporterVisitor.

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 27 13:10:24 PDT 2019


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


================
Comment at: clang/include/clang/StaticAnalyzer/Core/BugReporter/BugReporterVisitors.h:400
+
+  friend class Factory;
+  friend class TagVisitor;
----------------
xazax.hun wrote:
> NoQ wrote:
> > xazax.hun wrote:
> > > Isn't this redundant?
> > It isn't - i made a private constructor as usual.
> I had the impression that nested classes can access the members of their parent classes since C++11.
Wait, seriously?!~ Thx!


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

https://reviews.llvm.org/D58367





More information about the cfe-commits mailing list