| commit | 31b35a5641bc54a2ff1809b932323c3cff6b9f3d | [log] [tgz] |
|---|---|---|
| author | Daniel Cheng <[email protected]> | Mon Apr 23 10:34:55 2018 |
| committer | Commit Bot <[email protected]> | Mon Apr 23 10:34:55 2018 |
| tree | d57018aaf2d71b62f62c8443ebdce9af6296f44f | |
| parent | 39381c11e4c37f93bab5ada94a85730c70be8105 [diff] [blame] |
base/version.cc: move ownership of components vector argument in ctor. This is passed by value, so use std::move() as well. Change-Id: Ifd7518f3c2787b07dcd833d3b8943799e1d2ec5d Reviewed-on: https://chromium-review.googlesource.com/1023830 Reviewed-by: Nico Weber <[email protected]> Commit-Queue: Daniel Cheng <[email protected]> Cr-Commit-Position: refs/heads/master@{#552662}
diff --git a/base/version.cc b/base/version.cc index 7e897b2..3a54607 100644 --- a/base/version.cc +++ b/base/version.cc
@@ -91,7 +91,8 @@ components_.swap(parsed); } -Version::Version(std::vector<uint32_t> components) : components_(components) {} +Version::Version(std::vector<uint32_t> components) + : components_(std::move(components)) {} bool Version::IsValid() const { return (!components_.empty());