diff options
author | Jacopo Mondi <jacopo@jmondi.org> | 2020-09-02 12:11:46 +0200 |
---|---|---|
committer | Jacopo Mondi <jacopo@jmondi.org> | 2020-09-18 11:31:56 +0200 |
commit | facadb188efa32e0846fb59b73d8d669b7686e29 (patch) | |
tree | c52e6bc84795c37aec130442d153c1879f2683e3 /test/media_device/media_device_link_test.cpp | |
parent | c82f944399fd4ad81a440e3f852ee0fff876ba37 (diff) |
android: camera_device: Set Encoder at construction
Make the CameraStream encoder a private unique pointer and require its
initialization at construction time. This ties the encoder lifetime to
the CameraStream it has been created with, allowing to remove the
CameraStream destructor.
This change dis-allow creating a CameraStream and set the Encoder later,
which shall not happen now that we create CameraStream once we have all
the required information in place.
No functional changes intended but this change aims to make the code
more robust enforcing a stricter CameraStream interface.
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Niklas Söderlund <niklas.soderlund@ragnatech.se>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
Diffstat (limited to 'test/media_device/media_device_link_test.cpp')
0 files changed, 0 insertions, 0 deletions