diff options
author | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2019-01-27 15:39:26 +0100 |
---|---|---|
committer | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2019-01-31 11:03:21 +0200 |
commit | 92d9df69243000082559084a6fba2929cc18cc83 (patch) | |
tree | 1d6c2e87d380749574545cbf70fb1b2ee8a215a9 | |
parent | 5745bd3df519ed3064b2e8d308b90834272504b8 (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.h | 2 | ||||
-rw-r--r-- | include/libcamera/camera_manager.h | 2 |
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_; |