projects / platform / upstream / llvm.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: ff6ac1e) | patch
Removed some questionable default arguments from setters
authorDmitri Gribenko <gribozavr@gmail.com>
Thu, 12 Sep 2019 12:16:43 +0000 (12:16 +0000)
committerDmitri Gribenko <gribozavr@gmail.com>
Thu, 12 Sep 2019 12:16:43 +0000 (12:16 +0000)
commit20f45ed6990458b8cc96fe2101fbbb7f90037dd7
tree84a18fa604b734b26755101daf62c5b5c4e095c5tree | snapshot
parentff6ac1eb5f304ab2b690cf24509680fd62cdf743commit | diff
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
clang/include/clang/Basic/Diagnostic.h diff | blob | history
clang/lib/ARCMigrate/ARCMT.cpp diff | blob | history
clang/lib/Frontend/FrontendActions.cpp diff | blob | history
clang/lib/Sema/Sema.cpp diff | blob | history
Domain: System / Toolchain;