Fix cpplint errors.
The only non-obvious change here is the switch from dynamic_cast to
static_cast in three cases of down-casting in UpdateAttempter.
dynamic_cast is banned by style, nor does it add any safety in this
particular case (subsequent code dereferences the result right away
without checking whether it's null).
BUG=None
TEST=None
Change-Id: I9d49b46362feaf9c6fa13b2715ebe9fe50308a9a
Reviewed-on: https://chromium-review.googlesource.com/207470
Tested-by: Gilad Arnold <garnold@chromium.org>
Reviewed-by: Alex Vakulenko <avakulenko@chromium.org>
Commit-Queue: Gilad Arnold <garnold@chromium.org>
diff --git a/update_manager/real_updater_provider.cc b/update_manager/real_updater_provider.cc
index 7863019..6445972 100644
--- a/update_manager/real_updater_provider.cc
+++ b/update_manager/real_updater_provider.cc
@@ -141,8 +141,10 @@
{update_engine::kUpdateStatusVerifying, Stage::kVerifying},
{update_engine::kUpdateStatusFinalizing, Stage::kFinalizing},
{update_engine::kUpdateStatusUpdatedNeedReboot, Stage::kUpdatedNeedReboot},
- {update_engine::kUpdateStatusReportingErrorEvent,
- Stage::kReportingErrorEvent},
+ { // NOLINT(whitespace/braces)
+ update_engine::kUpdateStatusReportingErrorEvent,
+ Stage::kReportingErrorEvent
+ },
{update_engine::kUpdateStatusAttemptingRollback, Stage::kAttemptingRollback},
};
@@ -310,8 +312,8 @@
};
// A variable returning the number of consecutive failed update checks.
-class ConsecutiveFailedUpdateChecksVariable :
- public UpdaterVariableBase<unsigned int> {
+class ConsecutiveFailedUpdateChecksVariable
+ : public UpdaterVariableBase<unsigned int> {
public:
using UpdaterVariableBase<unsigned int>::UpdaterVariableBase;