| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
| |
Parameters of type std::nullptr_t can only have one value, so it doesn't make
sense to name them.
llvm-svn: 221340
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
testing::Unused parameters.
Reviewers: bkramer, klimek
Reviewed By: klimek
Subscribers: curdeius, cfe-commits
Differential Revision: http://reviews.llvm.org/D5479
llvm-svn: 218443
|
| |
|
|
| |
llvm-svn: 217036
|
| |
|
|
|
|
| |
overload.
llvm-svn: 216868
|
| |
|
|
|
|
| |
functions.
llvm-svn: 216718
|
| |
|
|
| |
llvm-svn: 214703
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
definition is visible.
Summary:
This allows us to copy the parameter name from the definition (as a comment)
or insert /*unused*/ in both places.
Reviewers: alexfh, klimek
Subscribers: cfe-commits
Differential Revision: http://reviews.llvm.org/D4772
llvm-svn: 214701
|
|
|
Summary:
We still allow the escape hatch foo(int /*x*/) and also suggest this
in a fixit. This is more powerful than the corresponding cpplint.py check
it also flags functions with multiple arguments as naming all arguments is
recommended by the google style guide.
Reviewers: alexfh, djasper
Subscribers: cfe-commits
Differential Revision: http://reviews.llvm.org/D4518
llvm-svn: 213075
|