fix trailing semis that various versions of clang format treat differently

Author: acmorrow

src/viam/examples/modules/complex/gizmo/api.cpp

@@ -24,12 +24,12 @@ API API::traits<Gizmo>::api() {
     return {"viam", "component", "gizmo"};
 }
 
-Gizmo::Gizmo(std::string name) : Component(std::move(name)){};
+Gizmo::Gizmo(std::string name) : Component(std::move(name)) {}
 
 /* Gizmo server methods */
 
 GizmoServer::GizmoServer(std::shared_ptr<ResourceManager> manager)
-    : ResourceServer(std::move(manager)){};
+    : ResourceServer(std::move(manager)) {}
 
 grpc::Status GizmoServer::DoOne(grpc::ServerContext* context,
                                 const DoOneRequest* request,
@@ -170,7 +170,7 @@ grpc::Status GizmoServer::DoTwo(::grpc::ServerContext* context,
 /* Gizmo client methods */
 
 GizmoClient::GizmoClient(std::string name, std::shared_ptr<grpc::Channel> channel)
-    : Gizmo(std::move(name)), stub_(GizmoService::NewStub(channel)), channel_(std::move(channel)){};
+    : Gizmo(std::move(name)), stub_(GizmoService::NewStub(channel)), channel_(std::move(channel)) {}
 
 bool GizmoClient::do_one(std::string arg1) {
     return make_client_helper(this, *stub_, &StubType::DoOne)

src/viam/examples/modules/complex/gizmo/impl.hpp

@@ -14,10 +14,10 @@ using namespace viam::sdk;
 // `validate` method that checks config validity.
 class MyGizmo : public Gizmo, public Reconfigurable {
    public:
-    MyGizmo(std::string name, std::string arg1) : Gizmo(std::move(name)), arg1_(std::move(arg1)){};
+    MyGizmo(std::string name, std::string arg1) : Gizmo(std::move(name)), arg1_(std::move(arg1)) {}
     MyGizmo(const Dependencies& deps, const ResourceConfig& cfg) : Gizmo(cfg.name()) {
         this->reconfigure(deps, cfg);
-    };
+    }
     void reconfigure(const Dependencies& deps, const ResourceConfig& cfg) override;
     static std::vector<std::string> validate(ResourceConfig cfg);
 

src/viam/examples/modules/complex/summation/api.cpp

@@ -24,12 +24,12 @@ API API::traits<Summation>::api() {
     return {"viam", "service", "summation"};
 }
 
-Summation::Summation(std::string name) : Service(std::move(name)){};
+Summation::Summation(std::string name) : Service(std::move(name)) {}
 
 /* Summation server methods */
 
 SummationServer::SummationServer(std::shared_ptr<ResourceManager> manager)
-    : ResourceServer(std::move(manager)){};
+    : ResourceServer(std::move(manager)) {}
 
 grpc::Status SummationServer::Sum(grpc::ServerContext* context,
                                   const SumRequest* request,
@@ -58,7 +58,7 @@ grpc::Status SummationServer::Sum(grpc::ServerContext* context,
 SummationClient::SummationClient(std::string name, std::shared_ptr<grpc::Channel> channel)
     : Summation(std::move(name)),
       stub_(SummationService::NewStub(channel)),
-      channel_(std::move(channel)){};
+      channel_(std::move(channel)) {};
 
 double SummationClient::sum(std::vector<double> numbers) {
     return make_client_helper(this, *stub_, &StubType::Sum)

src/viam/examples/modules/complex/summation/impl.hpp

@@ -13,10 +13,10 @@ using namespace viam::sdk;
 class MySummation : public Summation, public Reconfigurable {
    public:
     MySummation(std::string name, bool subtract)
-        : Summation(std::move(name)), subtract_(subtract){};
+        : Summation(std::move(name)), subtract_(subtract) {}
     MySummation(const Dependencies& deps, const ResourceConfig& cfg) : Summation(cfg.name()) {
         this->reconfigure(deps, cfg);
-    };
+    }
     void reconfigure(const Dependencies& deps, const ResourceConfig& cfg) override;
     static std::vector<std::string> validate(ResourceConfig cfg);
 

src/viam/sdk/components/base.cpp

@@ -24,7 +24,7 @@ bool operator==(const Base::properties& lhs, const Base::properties& rhs) {
            lhs.turning_radius_meters == rhs.turning_radius_meters;
 }
 
-Base::Base(std::string name) : Component(std::move(name)){};
+Base::Base(std::string name) : Component(std::move(name)) {}
 
 }  // namespace sdk
 }  // namespace viam

src/viam/sdk/components/board.cpp

@@ -16,7 +16,7 @@ API API::traits<Board>::api() {
     return {kRDK, kComponent, "board"};
 }
 
-Board::Board(std::string name) : Component(std::move(name)){};
+Board::Board(std::string name) : Component(std::move(name)) {}
 
 bool operator==(const Board::status& lhs, const Board::status& rhs) {
     return (lhs.analog_reader_values == rhs.analog_reader_values &&

src/viam/sdk/components/camera.cpp

@@ -130,7 +130,7 @@ std::string Camera::normalize_mime_type(const std::string& str) {
     return mime_type;
 }
 
-Camera::Camera(std::string name) : Component(std::move(name)){};
+Camera::Camera(std::string name) : Component(std::move(name)) {}
 
 bool operator==(const Camera::point_cloud& lhs, const Camera::point_cloud& rhs) {
     return lhs.mime_type == rhs.mime_type && lhs.pc == rhs.pc;

src/viam/sdk/components/encoder.cpp

@@ -15,7 +15,7 @@ API API::traits<Encoder>::api() {
     return {kRDK, kComponent, "encoder"};
 }
 
-Encoder::Encoder(std::string name) : Component(std::move(name)){};
+Encoder::Encoder(std::string name) : Component(std::move(name)) {}
 
 bool operator==(const Encoder::position& lhs, const Encoder::position& rhs) {
     return (lhs.value == rhs.value && lhs.type == rhs.type);

src/viam/sdk/components/generic.cpp

@@ -11,7 +11,7 @@ API API::traits<GenericComponent>::api() {
     return {kRDK, kComponent, "generic"};
 }
 
-GenericComponent::GenericComponent(std::string name) : Component(std::move(name)){};
+GenericComponent::GenericComponent(std::string name) : Component(std::move(name)) {}
 
 }  // namespace sdk
 }  // namespace viam

src/viam/sdk/components/motor.cpp

@@ -16,7 +16,7 @@ API API::traits<Motor>::api() {
     return {kRDK, kComponent, "motor"};
 }
 
-Motor::Motor(std::string name) : Component(std::move(name)){};
+Motor::Motor(std::string name) : Component(std::move(name)) {}
 
 bool operator==(const Motor::power_status& lhs, const Motor::power_status& rhs) {
     return (lhs.is_on == rhs.is_on && lhs.power_pct == rhs.power_pct);