[clang-tools-extra] 4b8accb - [clang-tidy][NFC] Fix readability-redundant-access-specifiers findings

Piotr Zegar via cfe-commits cfe-commits at lists.llvm.org
Sun Aug 27 01:53:40 PDT 2023


Author: Piotr Zegar
Date: 2023-08-27T08:52:10Z
New Revision: 4b8accbbf40367632f6e7185a553291c74089742

URL: https://github.com/llvm/llvm-project/commit/4b8accbbf40367632f6e7185a553291c74089742
DIFF: https://github.com/llvm/llvm-project/commit/4b8accbbf40367632f6e7185a553291c74089742.diff

LOG: [clang-tidy][NFC] Fix readability-redundant-access-specifiers findings

Fix issues found by clang-tidy in clang-tidy source directory.

Added: 
    

Modified: 
    clang-tools-extra/clang-tidy/ClangTidyOptions.h
    clang-tools-extra/clang-tidy/modernize/UseNullptrCheck.cpp
    clang-tools-extra/clang-tidy/readability/BracesAroundStatementsCheck.h

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/clang-tidy/ClangTidyOptions.h b/clang-tools-extra/clang-tidy/ClangTidyOptions.h
index e20a7d734a199c..e7636cb5d9b063 100644
--- a/clang-tools-extra/clang-tidy/ClangTidyOptions.h
+++ b/clang-tools-extra/clang-tidy/ClangTidyOptions.h
@@ -230,7 +230,6 @@ class FileOptionsBaseProvider : public DefaultOptionsProvider {
                           ClangTidyOptions OverrideOptions,
                           ConfigFileHandlers ConfigHandlers);
 
- protected:
   void addRawFileOptions(llvm::StringRef AbsolutePath,
                          std::vector<OptionsSource> &CurOptions);
 

diff  --git a/clang-tools-extra/clang-tidy/modernize/UseNullptrCheck.cpp b/clang-tools-extra/clang-tidy/modernize/UseNullptrCheck.cpp
index 47c647841d7605..d88db51025ed89 100644
--- a/clang-tools-extra/clang-tidy/modernize/UseNullptrCheck.cpp
+++ b/clang-tools-extra/clang-tidy/modernize/UseNullptrCheck.cpp
@@ -467,7 +467,6 @@ class CastSequenceVisitor : public RecursiveASTVisitor<CastSequenceVisitor> {
     llvm_unreachable("findContainingAncestor");
   }
 
-private:
   SourceManager &SM;
   ASTContext &Context;
   ArrayRef<StringRef> NullMacros;

diff  --git a/clang-tools-extra/clang-tidy/readability/BracesAroundStatementsCheck.h b/clang-tools-extra/clang-tidy/readability/BracesAroundStatementsCheck.h
index 9653dd30f078f4..249aa1aaaa9154 100644
--- a/clang-tools-extra/clang-tidy/readability/BracesAroundStatementsCheck.h
+++ b/clang-tools-extra/clang-tidy/readability/BracesAroundStatementsCheck.h
@@ -57,7 +57,6 @@ class BracesAroundStatementsCheck : public ClangTidyCheck {
     return TK_IgnoreUnlessSpelledInSource;
   }
 
-private:
   std::set<const Stmt *> ForceBracesStmts;
   const unsigned ShortStatementLines;
 };


        


More information about the cfe-commits mailing list