| commit | bb7211cd3762a8545d29b0d5bfebcb94ec4db081 | [log] [tgz] |
|---|---|---|
| author | Chris Watkins <[email protected]> | Wed Nov 29 07:16:38 2017 |
| committer | Commit Bot <[email protected]> | Wed Nov 29 07:16:38 2017 |
| tree | 47b0d5af3415c6d9b867f65eddf7a747e51726a1 | |
| parent | 32d039c92b61ceea12d36e6b69906af52b3989f1 [diff] [blame] |
Run clang-tidy modernize-use-equals-{delete,default} on //base
See the bugs and cxx post for justification and details:
https://groups.google.com/a/chromium.org/forum/#!topic/cxx/RkOHzIK6Tq8
This change was done using clang-tidy as described here:
https://chromium.googlesource.com/chromium/src/+/lkcr/docs/clang_tidy.md
In some cases the the tool leaves behind a string of commas where it
replaced a member initializer list
(https://bugs.llvm.org/show_bug.cgi?id=35051). They were cleaned up with:
git diff --name-only | \
xargs sed -E -i 's/(^\s*|\)\s*):[ ,]*= default/\1 = default/'
BUG=778959,778957
Change-Id: I95a3f3dae4796e4a7beb77793a6101f433a2c649
Reviewed-on: https://chromium-review.googlesource.com/789718
Reviewed-by: Daniel Cheng <[email protected]>
Commit-Queue: Chris Watkins <[email protected]>
Cr-Commit-Position: refs/heads/master@{#520033}diff --git a/base/sequenced_task_runner.cc b/base/sequenced_task_runner.cc index 4f36dc4..86771c67 100644 --- a/base/sequenced_task_runner.cc +++ b/base/sequenced_task_runner.cc
@@ -28,8 +28,7 @@ : task_runner_(std::move(task_runner)) { } -OnTaskRunnerDeleter::~OnTaskRunnerDeleter() { -} +OnTaskRunnerDeleter::~OnTaskRunnerDeleter() = default; OnTaskRunnerDeleter::OnTaskRunnerDeleter(OnTaskRunnerDeleter&&) = default;