summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>2019-01-27 15:39:26 +0100
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>2019-01-31 11:03:21 +0200
commit92d9df69243000082559084a6fba2929cc18cc83 (patch)
tree1d6c2e87d380749574545cbf70fb1b2ee8a215a9
parent5745bd3df519ed3064b2e8d308b90834272504b8 (diff)
libcamera: camera: Fix operator= definition
The class assignment operator is usually defined as returning a reference to the object, to allow a = b = c statements. While the return type makes no difference when deleting the operator in a class definition, it's still a good practice to use the correct return type. Fix it in the Camera and CameraManager classes. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
-rw-r--r--include/libcamera/camera.h2
-rw-r--r--include/libcamera/camera_manager.h2
2 files changed, 2 insertions, 2 deletions
diff --git a/include/libcamera/camera.h b/include/libcamera/camera.h
index a2ded62d..5ad8a09f 100644
--- a/include/libcamera/camera.h
+++ b/include/libcamera/camera.h
@@ -23,7 +23,7 @@ public:
const std::string &name);
Camera(const Camera &) = delete;
- void operator=(const Camera &) = delete;
+ Camera &operator=(const Camera &) = delete;
const std::string &name() const;
diff --git a/include/libcamera/camera_manager.h b/include/libcamera/camera_manager.h
index 56dbd26f..cf3a85ae 100644
--- a/include/libcamera/camera_manager.h
+++ b/include/libcamera/camera_manager.h
@@ -38,7 +38,7 @@ public:
private:
CameraManager();
CameraManager(const CameraManager &) = delete;
- void operator=(const CameraManager &) = delete;
+ CameraManager &operator=(const CameraManager &) = delete;
~CameraManager();
std::unique_ptr<DeviceEnumerator> enumerator_;