diff options
| author | Alex Vakulenko <avakulenko@chromium.org> | 2014-08-11 15:59:58 -0700 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-08-12 23:10:06 +0000 |
| commit | 157fe307f162d13f53e7f98f6e2da7c60c0ff001 (patch) | |
| tree | 85d3063f0b5b58c4aeb5fd9c1e7f492888454974 /update_manager/default_policy.h | |
| parent | fd45a731d9f9176ce134b34e2a84acc0cf403d1d (diff) | |
platform2: sweep the lint errors identified by the updated linter
cpplint.py has been updated and identified new issues in existing
code. Stuff like overridden functions that specify 'override' should
not be marked as 'virtual', and constructors with no parameters
should not be marked as 'explicit'.
BUG=None
TEST=cpplint.py `find ./platform2 -name *.cc -or -name *.h`
Change-Id: Ibb9de43286d874d076ffd5ebb1b13c36ec794f01
Reviewed-on: https://chromium-review.googlesource.com/211950
Reviewed-by: Alex Vakulenko <avakulenko@chromium.org>
Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>
Tested-by: Alex Vakulenko <avakulenko@chromium.org>
Diffstat (limited to 'update_manager/default_policy.h')
| -rw-r--r-- | update_manager/default_policy.h | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/update_manager/default_policy.h b/update_manager/default_policy.h index 2fb1a06b..526476ca 100644 --- a/update_manager/default_policy.h +++ b/update_manager/default_policy.h @@ -52,11 +52,11 @@ class DefaultPolicy : public Policy { virtual ~DefaultPolicy() {} // Policy overrides. - virtual EvalStatus UpdateCheckAllowed( + EvalStatus UpdateCheckAllowed( EvaluationContext* ec, State* state, std::string* error, UpdateCheckParams* result) const override; - virtual EvalStatus UpdateCanStart( + EvalStatus UpdateCanStart( EvaluationContext* ec, State* state, std::string* error, @@ -73,7 +73,7 @@ class DefaultPolicy : public Policy { return EvalStatus::kSucceeded; } - virtual EvalStatus UpdateDownloadAllowed( + EvalStatus UpdateDownloadAllowed( EvaluationContext* ec, State* state, std::string* error, @@ -84,9 +84,7 @@ class DefaultPolicy : public Policy { protected: // Policy override. - virtual std::string PolicyName() const override { - return "DefaultPolicy"; - } + std::string PolicyName() const override { return "DefaultPolicy"; } private: // A clock interface. |