| commit | 414abe7c7b3c2d2b027dad0fa04a074fd13c9852 | [log] [tgz] |
|---|---|---|
| author | Chris Watkins <watk@chromium.org> | Mon Nov 27 18:00:47 2017 +0000 |
| committer | Commit Bot <commit-bot@chromium.org> | Mon Nov 27 18:00:47 2017 +0000 |
| tree | 88e67f1099c03f157a0f8ec0e274d44fb47afcfa | |
| parent | 098696cbdca08d01b9b615cdec714dc7b34beffa [diff] |
Run clang-tidy modernize-use-equals-{delete,default} on //tools
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: Ib8d800da04575b9485affad6249ccb4736fc5dbe
Reviewed-on: https://chromium-review.googlesource.com/789746
Reviewed-by: Scott Graham <scottmg@chromium.org>
Commit-Queue: Scott Graham <scottmg@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#519336}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: e4963ba3016a4961fda1fbab0516b35ebe1af3da