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>

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__ */

include/ipa/ipa_vimc.h

@@ -17,6 +17,6 @@ enum IPAOperationCode {
 	IPAOperationInit,
 };
 
-}; /* namespace libcamera */
+} /* namespace libcamera */
 
 #endif /* __LIBCAMERA_IPA_VIMC_H__ */

include/libcamera/bound_method.h

@@ -142,6 +142,6 @@ private:
 	void (*func_)(Args...);
 };
 
-}; /* namespace libcamera */
+} /* namespace libcamera */
 
 #endif /* __LIBCAMERA_BOUND_METHOD_H__ */

include/libcamera/object.h

@@ -67,6 +67,6 @@ private:
 	unsigned int pendingMessages_;
 };
 
-}; /* namespace libcamera */
+} /* namespace libcamera */
 
 #endif /* __LIBCAMERA_OBJECT_H__ */

src/cam/event_loop.h

@@ -13,7 +13,7 @@
 
 namespace libcamera {
 class EventDispatcher;
-};
+}
 
 class EventLoop
 {

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)

src/ipa/ipa_vimc.cpp

@@ -112,6 +112,6 @@ IPAInterface *ipaCreate()
 {
 	return new IPAVimc();
 }
-};
+}
 
-}; /* namespace libcamera */
+} /* namespace libcamera */

src/ipa/rkisp1/rkisp1.cpp

@@ -251,6 +251,6 @@ IPAInterface *ipaCreate()
 {
 	return new IPARkISP1();
 }
-};
+}
 
-}; /* namespace libcamera */
+} /* namespace libcamera */

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;
 

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);
 

src/libcamera/message.cpp

@@ -161,4 +161,4 @@ void InvokeMessage::invoke()
  * \brief The packed method invocation arguments
  */
 
-}; /* namespace libcamera */
+} /* namespace libcamera */

src/libcamera/object.cpp

@@ -223,4 +223,4 @@ void Object::disconnect(SignalBase *signal)
 	}
 }
 
-}; /* namespace libcamera */
+} /* namespace libcamera */

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)

src/libcamera/proxy/ipa_proxy_linux.cpp

@@ -92,4 +92,4 @@ void IPAProxyLinux::readyRead(IPCUnixSocket *ipc)
 
 REGISTER_IPA_PROXY(IPAProxyLinux)
 
-}; /* namespace libcamera */
+} /* namespace libcamera */

src/libcamera/thread.cpp

@@ -495,4 +495,4 @@ void Thread::moveObject(Object *object, ThreadData *currentData,
 		moveObject(child, currentData, targetData);
 }
 
-}; /* namespace libcamera */
+} /* namespace libcamera */

test/camera/camera_test.cpp

@@ -45,4 +45,4 @@ void CameraTest::cleanup()
 
 	cm_->stop();
 	delete cm_;
-};
+}

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;

test/v4l2_videodevice/v4l2_videodevice_test.cpp

@@ -99,4 +99,4 @@ void V4L2VideoDeviceTest::cleanup()
 	delete debayer_;
 	delete sensor_;
 	delete capture_;
-};
+}