simple-cam: Use a unique_ptr for the CameraManager

The CameraManager should be released when it is no longer used. A
unique_ptr will handle this automatically, and convey the lifetime of
the object.

Update simple-cam to show that managing the lifetime of the
CameraManager is recommended practice.

Reviewed-by: Jacopo Mondi <jacopo@jmondi.org>
Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>

1 files changed, 8 insertions, 1 deletions

diff --git a/simple-cam.cpp b/simple-cam.cpp
index 8f7012b..5c16db6 100644
--- a/simple-cam.cpp
+++ b/simple-cam.cpp
@@ -131,8 +131,15 @@ int main()
 	 *
 	 * The CameraManager provides a list of available Cameras that
 	 * applications can operate on.
+	 *
+	 * When the CameraManager is no longer to be used, it should be deleted.
+	 * We use a unique_ptr here to manage the lifetime automatically during
+	 * the scope of this function.
+	 *
+	 * There can only be a single CameraManager constructed within any
+	 * process space.
 	 */
-	CameraManager *cm = new CameraManager();
+	std::unique_ptr<CameraManager> cm = std::make_unique<CameraManager>();
 	cm->start();
 
 	/*