| commit | 6fe52aa20e9426e16aa7204b0c5426895eb1c629 | [log] [tgz] |
|---|---|---|
| author | Chris Watkins <[email protected]> | Tue Nov 28 03:24:05 2017 |
| committer | Commit Bot <[email protected]> | Tue Nov 28 03:24:05 2017 |
| tree | e2e1f359b8795112d19071351a107f45c1debd6d | |
| parent | bcfb895bd39c421db37531d48accf340b4c0440f [diff] [blame] |
Run clang-tidy modernize-use-equals-{delete,default} on //remoting
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: I3805211c70d2c2340afa4d6d956339d1b25751fc
Reviewed-on: https://chromium-review.googlesource.com/789728
Reviewed-by: Sergey Ulanov <[email protected]>
Commit-Queue: Chris Watkins <[email protected]>
Cr-Commit-Position: refs/heads/master@{#519543}diff --git a/remoting/protocol/client_video_stats_dispatcher.cc b/remoting/protocol/client_video_stats_dispatcher.cc index 19b49ba..12e76d80 100644 --- a/remoting/protocol/client_video_stats_dispatcher.cc +++ b/remoting/protocol/client_video_stats_dispatcher.cc
@@ -25,7 +25,7 @@ : ChannelDispatcherBase(kVideoStatsChannelNamePrefix + stream_name), video_stats_stub_(video_stats_stub) {} -ClientVideoStatsDispatcher::~ClientVideoStatsDispatcher() {} +ClientVideoStatsDispatcher::~ClientVideoStatsDispatcher() = default; void ClientVideoStatsDispatcher::OnIncomingMessage( std::unique_ptr<CompoundBuffer> message) {