[build] Upgrade CI to clang-format 10.0 (#1961)

MacOS no longer ships 6.0, and Arch Linux's mesa GPU drivers are no longer compatible with LLVM 6.0.

wpilibNewCommands/src/main/native/cpp/frc2/command/CommandBase.cpp

@@ -51,14 +51,16 @@ void CommandBase::SetSubsystem(const wpi::Twine& subsystem) {
 
 void CommandBase::InitSendable(frc::SendableBuilder& builder) {
   builder.SetSmartDashboardType("Command");
-  builder.AddStringProperty(".name", [this] { return GetName(); }, nullptr);
-  builder.AddBooleanProperty("running", [this] { return IsScheduled(); },
-                             [this](bool value) {
-                               bool isScheduled = IsScheduled();
-                               if (value && !isScheduled) {
-                                 Schedule();
-                               } else if (!value && isScheduled) {
-                                 Cancel();
-                               }
-                             });
+  builder.AddStringProperty(
+      ".name", [this] { return GetName(); }, nullptr);
+  builder.AddBooleanProperty(
+      "running", [this] { return IsScheduled(); },
+      [this](bool value) {
+        bool isScheduled = IsScheduled();
+        if (value && !isScheduled) {
+          Schedule();
+        } else if (!value && isScheduled) {
+          Cancel();
+        }
+      });
 }

wpilibNewCommands/src/main/native/cpp/frc2/command/PIDCommand.cpp

@@ -37,15 +37,17 @@ PIDCommand::PIDCommand(PIDController controller,
                        std::function<double()> measurementSource,
                        double setpoint, std::function<void(double)> useOutput,
                        std::initializer_list<Subsystem*> requirements)
-    : PIDCommand(controller, measurementSource, [setpoint] { return setpoint; },
-                 useOutput, requirements) {}
+    : PIDCommand(
+          controller, measurementSource, [setpoint] { return setpoint; },
+          useOutput, requirements) {}
 
 PIDCommand::PIDCommand(PIDController controller,
                        std::function<double()> measurementSource,
                        double setpoint, std::function<void(double)> useOutput,
                        wpi::ArrayRef<Subsystem*> requirements)
-    : PIDCommand(controller, measurementSource, [setpoint] { return setpoint; },
-                 useOutput, requirements) {}
+    : PIDCommand(
+          controller, measurementSource, [setpoint] { return setpoint; },
+          useOutput, requirements) {}
 
 void PIDCommand::Initialize() { m_controller.Reset(); }
 

wpilibNewCommands/src/main/native/cpp/frc2/command/SubsystemBase.cpp

@@ -1,5 +1,5 @@
 /*----------------------------------------------------------------------------*/
-/* Copyright (c) 2019 FIRST. All Rights Reserved.                             */
+/* Copyright (c) 2019-2020 FIRST. All Rights Reserved.                        */
 /* Open Source Software - may be modified and shared by FRC teams. The code   */
 /* must be accompanied by the FIRST BSD license file in the root directory of */
 /* the project.                                                               */
@@ -22,26 +22,28 @@ SubsystemBase::SubsystemBase() {
 
 void SubsystemBase::InitSendable(frc::SendableBuilder& builder) {
   builder.SetSmartDashboardType("Subsystem");
-  builder.AddBooleanProperty(".hasDefault",
-                             [this] { return GetDefaultCommand() != nullptr; },
-                             nullptr);
-  builder.AddStringProperty(".default",
-                            [this]() -> std::string {
-                              auto command = GetDefaultCommand();
-                              if (command == nullptr) return "none";
-                              return command->GetName();
-                            },
-                            nullptr);
-  builder.AddBooleanProperty(".hasCommand",
-                             [this] { return GetCurrentCommand() != nullptr; },
-                             nullptr);
-  builder.AddStringProperty(".command",
-                            [this]() -> std::string {
-                              auto command = GetCurrentCommand();
-                              if (command == nullptr) return "none";
-                              return command->GetName();
-                            },
-                            nullptr);
+  builder.AddBooleanProperty(
+      ".hasDefault", [this] { return GetDefaultCommand() != nullptr; },
+      nullptr);
+  builder.AddStringProperty(
+      ".default",
+      [this]() -> std::string {
+        auto command = GetDefaultCommand();
+        if (command == nullptr) return "none";
+        return command->GetName();
+      },
+      nullptr);
+  builder.AddBooleanProperty(
+      ".hasCommand", [this] { return GetCurrentCommand() != nullptr; },
+      nullptr);
+  builder.AddStringProperty(
+      ".command",
+      [this]() -> std::string {
+        auto command = GetCurrentCommand();
+        if (command == nullptr) return "none";
+        return command->GetName();
+      },
+      nullptr);
 }
 
 std::string SubsystemBase::GetName() const {

wpilibNewCommands/src/main/native/include/frc2/command/ProfiledPIDCommand.h

@@ -96,11 +96,12 @@ class ProfiledPIDCommand
                      std::function<Distance_t()> goalSource,
                      std::function<void(double, State)> useOutput,
                      std::initializer_list<Subsystem*> requirements)
-      : ProfiledPIDCommand(controller, measurementSource,
-                           [&goalSource]() {
-                             return State{goalSource(), Velocity_t{0}};
-                           },
-                           useOutput, requirements) {}
+      : ProfiledPIDCommand(
+            controller, measurementSource,
+            [&goalSource]() {
+              return State{goalSource(), Velocity_t{0}};
+            },
+            useOutput, requirements) {}
 
   /**
    * Creates a new PIDCommand, which controls the given output with a
@@ -117,11 +118,12 @@ class ProfiledPIDCommand
                      std::function<Distance_t()> goalSource,
                      std::function<void(double, State)> useOutput,
                      wpi::ArrayRef<Subsystem*> requirements = {})
-      : ProfiledPIDCommand(controller, measurementSource,
-                           [&goalSource]() {
-                             return State{goalSource(), Velocity_t{0}};
-                           },
-                           useOutput, requirements) {}
+      : ProfiledPIDCommand(
+            controller, measurementSource,
+            [&goalSource]() {
+              return State{goalSource(), Velocity_t{0}};
+            },
+            useOutput, requirements) {}
 
   /**
    * Creates a new PIDCommand, which controls the given output with a
@@ -137,8 +139,9 @@ class ProfiledPIDCommand
                      std::function<Distance_t()> measurementSource, State goal,
                      std::function<void(double, State)> useOutput,
                      std::initializer_list<Subsystem*> requirements)
-      : ProfiledPIDCommand(controller, measurementSource,
-                           [goal] { return goal; }, useOutput, requirements) {}
+      : ProfiledPIDCommand(
+            controller, measurementSource, [goal] { return goal; }, useOutput,
+            requirements) {}
 
   /**
    * Creates a new PIDCommand, which controls the given output with a
@@ -154,8 +157,9 @@ class ProfiledPIDCommand
                      std::function<Distance_t()> measurementSource, State goal,
                      std::function<void(double, State)> useOutput,
                      wpi::ArrayRef<Subsystem*> requirements = {})
-      : ProfiledPIDCommand(controller, measurementSource,
-                           [goal] { return goal; }, useOutput, requirements) {}
+      : ProfiledPIDCommand(
+            controller, measurementSource, [goal] { return goal; }, useOutput,
+            requirements) {}
 
   /**
    * Creates a new PIDCommand, which controls the given output with a
@@ -172,8 +176,9 @@ class ProfiledPIDCommand
                      Distance_t goal,
                      std::function<void(double, State)> useOutput,
                      std::initializer_list<Subsystem*> requirements)
-      : ProfiledPIDCommand(controller, measurementSource,
-                           [goal] { return goal; }, useOutput, requirements) {}
+      : ProfiledPIDCommand(
+            controller, measurementSource, [goal] { return goal; }, useOutput,
+            requirements) {}
 
   /**
    * Creates a new PIDCommand, which controls the given output with a
@@ -190,8 +195,9 @@ class ProfiledPIDCommand
                      Distance_t goal,
                      std::function<void(double, State)> useOutput,
                      wpi::ArrayRef<Subsystem*> requirements = {})
-      : ProfiledPIDCommand(controller, measurementSource,
-                           [goal] { return goal; }, useOutput, requirements) {}
+      : ProfiledPIDCommand(
+            controller, measurementSource, [goal] { return goal; }, useOutput,
+            requirements) {}
 
   ProfiledPIDCommand(ProfiledPIDCommand&& other) = default;