| commit | 7a41d355a4a582e2139e21d58ce08fb423092ab7 | [log] [tgz] |
|---|---|---|
| author | Chris Watkins <[email protected]> | Fri Dec 01 02:13:27 2017 |
| committer | Commit Bot <[email protected]> | Fri Dec 01 02:13:27 2017 |
| tree | 6ceb879108a528dd878ca0040d7a28e52a19df32 | |
| parent | ef56ceb8f8c05e593428821371f1407908f78624 [diff] [blame] |
Run clang-tidy modernize-use-equals-{delete,default} on //net, pt 2
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
This CL was uploaded by git cl split.
[email protected]
Change-Id: I0c2a41fbeee3648e886b70f259e333176ffa4334
Reviewed-on: https://chromium-review.googlesource.com/799595
Reviewed-by: Eric Roman <[email protected]>
Commit-Queue: Chris Watkins <[email protected]>
Cr-Commit-Position: refs/heads/master@{#520798}diff --git a/net/socket/ssl_server_socket_unittest.cc b/net/socket/ssl_server_socket_unittest.cc index 102e8033b..73a038d 100644 --- a/net/socket/ssl_server_socket_unittest.cc +++ b/net/socket/ssl_server_socket_unittest.cc
@@ -222,7 +222,7 @@ FakeDataChannel* outgoing_channel) : incoming_(incoming_channel), outgoing_(outgoing_channel) {} - ~FakeSocket() override {} + ~FakeSocket() override = default; int Read(IOBuffer* buf, int buf_len,