diff options
| author | Dmitri Gribenko <gribozavr@gmail.com> | 2019-09-12 12:16:43 +0000 |
|---|---|---|
| committer | Dmitri Gribenko <gribozavr@gmail.com> | 2019-09-12 12:16:43 +0000 |
| commit | 20f45ed6990458b8cc96fe2101fbbb7f90037dd7 (patch) | |
| tree | 84a18fa604b734b26755101daf62c5b5c4e095c5 /clang/lib/Frontend/FrontendActions.cpp | |
| parent | ff6ac1eb5f304ab2b690cf24509680fd62cdf743 (diff) | |
| download | bcm5719-llvm-20f45ed6990458b8cc96fe2101fbbb7f90037dd7.tar.gz bcm5719-llvm-20f45ed6990458b8cc96fe2101fbbb7f90037dd7.zip | |
Removed some questionable default arguments from setters
Summary:
They can be confusing -- what does it mean to call a setter without a
value? Also, some setters, like `setPrintTemplateTree` had `false` as
the default value!
The callers are largely not using these default arguments anyway.
Subscribers: cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D67491
llvm-svn: 371731
Diffstat (limited to 'clang/lib/Frontend/FrontendActions.cpp')
| -rw-r--r-- | clang/lib/Frontend/FrontendActions.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/lib/Frontend/FrontendActions.cpp b/clang/lib/Frontend/FrontendActions.cpp index 28416e72267..dc830cbacbd 100644 --- a/clang/lib/Frontend/FrontendActions.cpp +++ b/clang/lib/Frontend/FrontendActions.cpp @@ -928,7 +928,7 @@ void PrintDependencyDirectivesSourceMinimizerAction::ExecuteAction() { // 'expected' comments. if (CI.getDiagnosticOpts().VerifyDiagnostics) { // Make sure we don't emit new diagnostics! - CI.getDiagnostics().setSuppressAllDiagnostics(); + CI.getDiagnostics().setSuppressAllDiagnostics(true); Preprocessor &PP = getCompilerInstance().getPreprocessor(); PP.EnterMainSourceFile(); Token Tok; |
