[libcamera-devel] [PATCH 03/10] libcamera: controls: Prevent copies of ControlId class

Laurent Pinchart laurent.pinchart at ideasonboard.com
Mon Oct 14 01:27:49 CEST 2019


Now that the V4L2ControlId embedded in V4L2ControlInfo has been moved
out and doesn't require to be copied anymore, the base ControlId class
doesn't need to be copyable either. Delete the ControlId copy
constructor and copy assignment operator.

Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
---
 include/libcamera/controls.h | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/include/libcamera/controls.h b/include/libcamera/controls.h
index ebc4204f98fd..5534a2edb567 100644
--- a/include/libcamera/controls.h
+++ b/include/libcamera/controls.h
@@ -68,12 +68,11 @@ protected:
 		: id_(id), name_(name), type_(type)
 	{
 	}
-#ifndef __DOXYGEN__
-	ControlId &operator=(const ControlId &) = default;
-	ControlId(const ControlId &) = default;
-#endif
 
 private:
+	ControlId &operator=(const ControlId &) = delete;
+	ControlId(const ControlId &) = delete;
+
 	unsigned int id_;
 	std::string name_;
 	ControlType type_;
-- 
Regards,

Laurent Pinchart



More information about the libcamera-devel mailing list