| commit | c4e452f35f51846abd70634b4782d0d52900a82d | [log] [tgz] |
|---|---|---|
| author | Chris Watkins <watk@chromium.org> | Wed Nov 29 07:44:11 2017 |
| committer | Commit Bot <commit-bot@chromium.org> | Wed Nov 29 07:44:11 2017 |
| tree | c1b59b335a5b06645122f5d036d034a8711ec613 | |
| parent | 3e60342e82dfebc356d20bcd1d1930593adda818 [diff] |
Run clang-tidy modernize-use-equals-{delete,default} on //dbus
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: I3435f847095ac0fbe5532f7859a67acdb4b23e8c
Reviewed-on: https://chromium-review.googlesource.com/789741
Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Commit-Queue: Chris Watkins <watk@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#520038}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 3740aae9d73f5769c51c9be67150e619378167b3