diff options
author | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2019-04-29 20:58:34 +0300 |
---|---|---|
committer | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2019-05-23 01:07:21 +0300 |
commit | 75de3bff9e165e512be5f53e19f31c7388dc08b1 (patch) | |
tree | bc91c3263a613fc492820af4d753147e971df65d /src/qcam/main_window.h | |
parent | 77ae64eb24bf6c2fe241b6f7b841315b1ee15707 (diff) |
libcamera: camera: Return a pointer from generateConfiguration()
To prepare for specialising the CameraConfiguration class in pipeline
handlers, return a pointer to a camera configuration instead of a
reference from Camera::generateConfiguration(). The camera configuration
always needs to be allocated from the pipeline handler, and its
ownership is passed to the application.
For symmetry, change Camera::configure() to take a CameraConfiguration
pointer instead of a reference. This aligns with our coding practice of
passing parameters that are modified by the callee by pointer.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Niklas Söderlund <niklas.soderlund@ragnatech.se>
Reviewed-by: Jacopo Mondi <jacopo@jmondi.org>
Diffstat (limited to 'src/qcam/main_window.h')
-rw-r--r-- | src/qcam/main_window.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/qcam/main_window.h b/src/qcam/main_window.h index 143b5b08..fe565cbc 100644 --- a/src/qcam/main_window.h +++ b/src/qcam/main_window.h @@ -8,6 +8,7 @@ #define __QCAM_MAIN_WINDOW_H__ #include <map> +#include <memory> #include <QMainWindow> @@ -45,7 +46,7 @@ private: std::shared_ptr<Camera> camera_; bool isCapturing_; - CameraConfiguration config_; + std::unique_ptr<CameraConfiguration> config_; ViewFinder *viewfinder_; }; |