commit 790cb54590e4f28f61e2f1bcd2e6e12ca47d7713
author Philipp Schrader <philipp.schrader@gmail.com> Wed Jul 05 21:06:52 2023 -0700
committer Philipp Schrader <philipp.schrader@gmail.com> Tue Jul 18 22:20:04 2023 -0700
tree 1f709f6cde402b0f9f3cd62cc66d4e2d4c3faa16
parent ecd7e973ee4fbab474f746c4b719d493a933b5be

Run clang-format on the entire repo

This patch clang-formats the entire repo. Third-party code is
excluded.

I needed to fix up the .clang-format file so that all the header
includes are ordered properly. I could have sworn that it used to work
without the extra modification, but I guess not.

Signed-off-by: Philipp Schrader <philipp.schrader@gmail.com>
Change-Id: I64bb9f2c795401393f9dfe2fefc4f04cb36b52f6

y2022/localizer/localizer.cc

diff --git a/y2022/localizer/localizer.cc b/y2022/localizer/localizer.cc
index ec81573..b774e18 100644
--- a/y2022/localizer/localizer.cc
+++ b/y2022/localizer/localizer.cc
@@ -909,43 +909,43 @@
             absl::StrCat("/", kPisToUse[camera_index], "/camera"));
   }
   aos::TimerHandler *estimate_timer = event_loop_->AddTimer([this]() {
-      const bool maybe_in_auto = utils_.MaybeInAutonomous();
-      model_based_.set_use_aggressive_image_corrections(!maybe_in_auto);
-      for (size_t camera_index = 0; camera_index < kPisToUse.size();
-           ++camera_index) {
-        if (model_based_.NumQueuedImageDebugs() ==
-                ModelBasedLocalizer::kDebugBufferSize ||
-            (last_visualization_send_ + kMinVisualizationPeriod <
-             event_loop_->monotonic_now())) {
-          auto builder = visualization_sender_.MakeBuilder();
-          visualization_sender_.CheckOk(
-              builder.Send(model_based_.PopulateVisualization(builder.fbb())));
+    const bool maybe_in_auto = utils_.MaybeInAutonomous();
+    model_based_.set_use_aggressive_image_corrections(!maybe_in_auto);
+    for (size_t camera_index = 0; camera_index < kPisToUse.size();
+         ++camera_index) {
+      if (model_based_.NumQueuedImageDebugs() ==
+              ModelBasedLocalizer::kDebugBufferSize ||
+          (last_visualization_send_ + kMinVisualizationPeriod <
+           event_loop_->monotonic_now())) {
+        auto builder = visualization_sender_.MakeBuilder();
+        visualization_sender_.CheckOk(
+            builder.Send(model_based_.PopulateVisualization(builder.fbb())));
+      }
+      if (target_estimate_fetchers_[camera_index].Fetch()) {
+        const std::optional<aos::monotonic_clock::duration> monotonic_offset =
+            utils_.ClockOffset(kPisToUse[camera_index]);
+        if (!monotonic_offset.has_value()) {
+          model_based_.TallyRejection(
+              RejectionReason::MESSAGE_BRIDGE_DISCONNECTED);
+          continue;
         }
-        if (target_estimate_fetchers_[camera_index].Fetch()) {
-          const std::optional<aos::monotonic_clock::duration> monotonic_offset =
-              utils_.ClockOffset(kPisToUse[camera_index]);
-          if (!monotonic_offset.has_value()) {
-            model_based_.TallyRejection(
-                RejectionReason::MESSAGE_BRIDGE_DISCONNECTED);
-            continue;
-          }
-          // TODO(james): Get timestamp from message contents.
-          aos::monotonic_clock::time_point capture_time(
-              target_estimate_fetchers_[camera_index]
-                  .context()
-                  .monotonic_remote_time -
-              monotonic_offset.value());
-          if (capture_time > target_estimate_fetchers_[camera_index]
-                                 .context()
-                                 .monotonic_event_time) {
-            model_based_.TallyRejection(RejectionReason::IMAGE_FROM_FUTURE);
-            continue;
-          }
-          capture_time -= imu_watcher_.pico_offset_error();
-          model_based_.HandleImageMatch(
-              capture_time, target_estimate_fetchers_[camera_index].get(),
-              camera_index);
+        // TODO(james): Get timestamp from message contents.
+        aos::monotonic_clock::time_point capture_time(
+            target_estimate_fetchers_[camera_index]
+                .context()
+                .monotonic_remote_time -
+            monotonic_offset.value());
+        if (capture_time > target_estimate_fetchers_[camera_index]
+                               .context()
+                               .monotonic_event_time) {
+          model_based_.TallyRejection(RejectionReason::IMAGE_FROM_FUTURE);
+          continue;
         }
+        capture_time -= imu_watcher_.pico_offset_error();
+        model_based_.HandleImageMatch(
+            capture_time, target_estimate_fetchers_[camera_index].get(),
+            camera_index);
+      }
     }
   });
   event_loop_->OnRun([this, estimate_timer]() {
@@ -959,7 +959,6 @@
     aos::monotonic_clock::time_point sample_time_pi,
     std::optional<Eigen::Vector2d> encoders, Eigen::Vector3d gyro,
     Eigen::Vector3d accel) {
-
   model_based_.HandleImu(
       sample_time_pico, gyro, accel, encoders,
       utils_.VoltageOrZero(event_loop_->context().monotonic_event_time));