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

Gábor Horváth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 27 12:59:48 PDT 2019


xazax.hun added inline comments.


================
Comment at: clang/include/clang/StaticAnalyzer/Core/BugReporter/BugReporterVisitors.h:400
+
+  friend class Factory;
+  friend class TagVisitor;
----------------
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.


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

https://reviews.llvm.org/D58367





More information about the cfe-commits mailing list