[PATCH] D55793: [clang-tidy] Add duplicated access specifier readability check (PR25403)

[Roman Lebedev via Phabricator via cfe-commits]

lebedev.ri added a comment.

Please add tests with preprocessor (`#if ...`) that will show that it ignores 
disabled code. e.g.:

  class ProbablyValid {
  private:
    int a;
  #if defined(ZZ)
  public:
    int b;
  #endif
  private:
    int c;
  protected:
    int d;
  public:
    int e;
  };


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

https://reviews.llvm.org/D55793



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits