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/callback_unittest.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/callback_unittest.cc')
| -rw-r--r-- | base/callback_unittest.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/base/callback_unittest.cc b/base/callback_unittest.cc index 2c2bef1a..6103b2e 100644 --- a/base/callback_unittest.cc +++ b/base/callback_unittest.cc @@ -148,7 +148,7 @@ TEST_F(CallbackTest, ResetAndReturn) { class CallbackOwner : public base::RefCounted<CallbackOwner> { public: - CallbackOwner(bool* deleted) { + explicit CallbackOwner(bool* deleted) { callback_ = Bind(&CallbackOwner::Unused, this); deleted_ = deleted; } |