diff options
| author | hans@chromium.org <hans@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-01-15 10:52:11 +0000 |
|---|---|---|
| committer | hans@chromium.org <hans@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-01-15 10:52:11 +0000 |
| commit | f3c697c5e5bc6bf15f6e8846f189b46dfc1e12c5 (patch) | |
| tree | 1d7a6fc34e066dcbf9d2bc01af2244f72cbd78a7 /base/values.cc | |
| parent | d443be6fdfe17ca4f3ff1843ded362ff0cd01096 (diff) | |
| download | chromium_src-f3c697c5e5bc6bf15f6e8846f189b46dfc1e12c5.zip chromium_src-f3c697c5e5bc6bf15f6e8846f189b46dfc1e12c5.tar.gz chromium_src-f3c697c5e5bc6bf15f6e8846f189b46dfc1e12c5.tar.bz2 | |
Add missing, and remove unnecessary, 'explicit' from constructors. (base/)
The style guide says that constructors which can be called with one
argument should be explicit.
For constructors which cannot be called with exactly one argument,
there is no reason to mark them explicit.
BUG=163357
Review URL: https://chromiumcodereview.appspot.com/11779022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176881 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/values.cc')
| -rw-r--r-- | base/values.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/base/values.cc b/base/values.cc index 7a21709..4c8968f 100644 --- a/base/values.cc +++ b/base/values.cc @@ -68,7 +68,7 @@ class ValueEquals { // Pass the value against which all consecutive calls of the () operator will // compare their argument to. This Value object must not be destroyed while // the ValueEquals is in use. - ValueEquals(const Value* first) : first_(first) { } + explicit ValueEquals(const Value* first) : first_(first) { } bool operator ()(const Value* second) const { return first_->Equals(second); |