libcamera: Remove unneeded semicolons

Comply with the coding style by removing lots of unneeded semicolons.
Fix a few other coding style violations on the lines touched by those
fixes.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>

18 files changed, 23 insertions, 21 deletions

diff --git a/include/ipa/ipa_module_info.h b/include/ipa/ipa_module_info.h
index d9e33c1d..7ecd1495 100644
--- a/include/ipa/ipa_module_info.h
+++ b/include/ipa/ipa_module_info.h
@@ -23,8 +23,8 @@ struct IPAModuleInfo {
 
 extern "C" {
 extern const struct IPAModuleInfo ipaModuleInfo;
-};
+}
 
-}; /* namespace libcamera */
+} /* namespace libcamera */
 
 #endif /* __LIBCAMERA_IPA_MODULE_INFO_H__ */
diff --git a/include/ipa/ipa_vimc.h b/include/ipa/ipa_vimc.h
index 05b561d8..9add122c 100644
--- a/include/ipa/ipa_vimc.h
+++ b/include/ipa/ipa_vimc.h
@@ -17,6 +17,6 @@ enum IPAOperationCode {
 	IPAOperationInit,
 };
 
-}; /* namespace libcamera */
+} /* namespace libcamera */
 
 #endif /* __LIBCAMERA_IPA_VIMC_H__ */
diff --git a/include/libcamera/bound_method.h b/include/libcamera/bound_method.h
index bdeb5469..b841a2ae 100644
--- a/include/libcamera/bound_method.h
+++ b/include/libcamera/bound_method.h
@@ -142,6 +142,6 @@ private:
 	void (*func_)(Args...);
 };
 
-}; /* namespace libcamera */
+} /* namespace libcamera */
 
 #endif /* __LIBCAMERA_BOUND_METHOD_H__ */
diff --git a/include/libcamera/object.h b/include/libcamera/object.h
index 91b56596..86e0f726 100644
--- a/include/libcamera/object.h
+++ b/include/libcamera/object.h
@@ -67,6 +67,6 @@ private:
 	unsigned int pendingMessages_;
 };
 
-}; /* namespace libcamera */
+} /* namespace libcamera */
 
 #endif /* __LIBCAMERA_OBJECT_H__ */
diff --git a/src/cam/event_loop.h b/src/cam/event_loop.h
index aaca5838..581c7cba 100644
--- a/src/cam/event_loop.h
+++ b/src/cam/event_loop.h
@@ -13,7 +13,7 @@
 
 namespace libcamera {
 class EventDispatcher;
-};
+}
 
 class EventLoop
 {
diff --git a/src/cam/options.cpp b/src/cam/options.cpp
index 1f0631ec..7e2dfa63 100644
--- a/src/cam/options.cpp
+++ b/src/cam/options.cpp
@@ -490,7 +490,7 @@ void OptionsParser::usage()
 			else
 				argument += "  ";
 			argument += std::string("--") + option.name;
-		};
+		}
 
 		if (option.argument != ArgumentNone) {
 			if (option.argument == ArgumentOptional)
diff --git a/src/ipa/ipa_vimc.cpp b/src/ipa/ipa_vimc.cpp
index 63d578b4..9fd5212b 100644
--- a/src/ipa/ipa_vimc.cpp
+++ b/src/ipa/ipa_vimc.cpp
@@ -112,6 +112,6 @@ IPAInterface *ipaCreate()
 {
 	return new IPAVimc();
 }
-};
+}
 
-}; /* namespace libcamera */
+} /* namespace libcamera */
diff --git a/src/ipa/rkisp1/rkisp1.cpp b/src/ipa/rkisp1/rkisp1.cpp
index 9a13f5c7..d741d567 100644
--- a/src/ipa/rkisp1/rkisp1.cpp
+++ b/src/ipa/rkisp1/rkisp1.cpp
@@ -251,6 +251,6 @@ IPAInterface *ipaCreate()
 {
 	return new IPARkISP1();
 }
-};
+}
 
-}; /* namespace libcamera */
+} /* namespace libcamera */
diff --git a/src/libcamera/include/ipa_proxy.h b/src/libcamera/include/ipa_proxy.h
index 72216662..add40b4b 100644
--- a/src/libcamera/include/ipa_proxy.h
+++ b/src/libcamera/include/ipa_proxy.h
@@ -36,7 +36,7 @@ class IPAProxyFactory
 {
 public:
 	IPAProxyFactory(const char *name);
-	virtual ~IPAProxyFactory(){};
+	virtual ~IPAProxyFactory() {}
 
 	virtual std::unique_ptr<IPAProxy> create(IPAModule *ipam) = 0;
 
diff --git a/src/libcamera/include/pipeline_handler.h b/src/libcamera/include/pipeline_handler.h
index 42b90a4b..241af58b 100644
--- a/src/libcamera/include/pipeline_handler.h
+++ b/src/libcamera/include/pipeline_handler.h
@@ -111,7 +111,7 @@ class PipelineHandlerFactory
 {
 public:
 	PipelineHandlerFactory(const char *name);
-	virtual ~PipelineHandlerFactory() { };
+	virtual ~PipelineHandlerFactory() {}
 
 	std::shared_ptr<PipelineHandler> create(CameraManager *manager);
 
diff --git a/src/libcamera/message.cpp b/src/libcamera/message.cpp
index daf65322..c5478953 100644
--- a/src/libcamera/message.cpp
+++ b/src/libcamera/message.cpp
@@ -161,4 +161,4 @@ void InvokeMessage::invoke()
  * \brief The packed method invocation arguments
  */
 
-}; /* namespace libcamera */
+} /* namespace libcamera */
diff --git a/src/libcamera/object.cpp b/src/libcamera/object.cpp
index db2c6f68..1f787271 100644
--- a/src/libcamera/object.cpp
+++ b/src/libcamera/object.cpp
@@ -223,4 +223,4 @@ void Object::disconnect(SignalBase *signal)
 	}
 }
 
-}; /* namespace libcamera */
+} /* namespace libcamera */
diff --git a/src/libcamera/pipeline_handler.cpp b/src/libcamera/pipeline_handler.cpp
index bf8c86d3..884feaa6 100644
--- a/src/libcamera/pipeline_handler.cpp
+++ b/src/libcamera/pipeline_handler.cpp
@@ -135,7 +135,7 @@ PipelineHandler::~PipelineHandler()
 {
 	for (std::shared_ptr<MediaDevice> media : mediaDevices_)
 		media->release();
-};
+}
 
 /**
  * \fn PipelineHandler::match(DeviceEnumerator *enumerator)
diff --git a/src/libcamera/proxy/ipa_proxy_linux.cpp b/src/libcamera/proxy/ipa_proxy_linux.cpp
index 4e6fa689..27b6639d 100644
--- a/src/libcamera/proxy/ipa_proxy_linux.cpp
+++ b/src/libcamera/proxy/ipa_proxy_linux.cpp
@@ -92,4 +92,4 @@ void IPAProxyLinux::readyRead(IPCUnixSocket *ipc)
 
 REGISTER_IPA_PROXY(IPAProxyLinux)
 
-}; /* namespace libcamera */
+} /* namespace libcamera */
diff --git a/src/libcamera/thread.cpp b/src/libcamera/thread.cpp
index 872ad1bd..e152af14 100644
--- a/src/libcamera/thread.cpp
+++ b/src/libcamera/thread.cpp
@@ -495,4 +495,4 @@ void Thread::moveObject(Object *object, ThreadData *currentData,
 		moveObject(child, currentData, targetData);
 }
 
-}; /* namespace libcamera */
+} /* namespace libcamera */
diff --git a/test/camera/camera_test.cpp b/test/camera/camera_test.cpp
index 0e105414..101e31fb 100644
--- a/test/camera/camera_test.cpp
+++ b/test/camera/camera_test.cpp
@@ -45,4 +45,4 @@ void CameraTest::cleanup()
 
 	cm_->stop();
 	delete cm_;
-};
+}
diff --git a/test/v4l2_subdevice/v4l2_subdevice_test.h b/test/v4l2_subdevice/v4l2_subdevice_test.h
index 96646a15..3bce6691 100644
--- a/test/v4l2_subdevice/v4l2_subdevice_test.h
+++ b/test/v4l2_subdevice/v4l2_subdevice_test.h
@@ -21,7 +21,9 @@ class V4L2SubdeviceTest : public Test
 {
 public:
 	V4L2SubdeviceTest()
-		: scaler_(nullptr){};
+		: scaler_(nullptr)
+	{
+	}
 
 protected:
 	int init() override;
diff --git a/test/v4l2_videodevice/v4l2_videodevice_test.cpp b/test/v4l2_videodevice/v4l2_videodevice_test.cpp
index 096f9649..577da4cb 100644
--- a/test/v4l2_videodevice/v4l2_videodevice_test.cpp
+++ b/test/v4l2_videodevice/v4l2_videodevice_test.cpp
@@ -99,4 +99,4 @@ void V4L2VideoDeviceTest::cleanup()
 	delete debayer_;
 	delete sensor_;
 	delete capture_;
-};
+}