[libcamera-devel] [PATCH] android: Camera3RequestDescriptor: Provide a constructor for StreamBuffer

[Umang Jain]

Provide a constructor for StreamBuffer and use that while populating
Camera3RequestDescriptor::buffers_ vector. Also provide the default
move-constructor (required as StreamBuffer is stored in a vector in
Camera3RequestDescriptor) and destructor for the StreamBuffer struct.

Also initialize pointers members in the StreamBuffer struct to nullptr,
with StreamBuffer::status set to Status::Success.

Signed-off-by: Umang Jain <umang.jain at ideasonboard.com>
---
 src/android/camera_request.cpp | 16 +++++++++++++---
 src/android/camera_request.h   | 12 +++++++++---
 2 files changed, 22 insertions(+), 6 deletions(-)

diff --git a/src/android/camera_request.cpp b/src/android/camera_request.cpp
index 5bac1b8f..0b2946fb 100644
--- a/src/android/camera_request.cpp
+++ b/src/android/camera_request.cpp
@@ -37,9 +37,7 @@ Camera3RequestDescriptor::Camera3RequestDescriptor(
 		CameraStream *stream =
 			static_cast<CameraStream *>(buffer.stream->priv);
 
-		buffers_.push_back({ stream, buffer.buffer, nullptr,
-				     buffer.acquire_fence, Status::Success,
-				     nullptr, nullptr, nullptr, this });
+		buffers_.emplace_back(stream, buffer, this);
 	}
 
 	/* Clone the controls associated with the camera3 request. */
@@ -54,3 +52,15 @@ Camera3RequestDescriptor::Camera3RequestDescriptor(
 }
 
 Camera3RequestDescriptor::~Camera3RequestDescriptor() = default;
+
+Camera3RequestDescriptor::StreamBuffer::StreamBuffer(
+	CameraStream *stream, const camera3_stream_buffer_t &buffer,
+	Camera3RequestDescriptor *request)
+	: camera3Buffer(buffer.buffer), fence(buffer.acquire_fence),
+	  stream(stream), request(request)
+{
+}
+
+Camera3RequestDescriptor::StreamBuffer::~StreamBuffer() = default;
+
+Camera3RequestDescriptor::StreamBuffer::StreamBuffer(StreamBuffer &&other) = default;
diff --git a/src/android/camera_request.h b/src/android/camera_request.h
index cfafa445..0a744aca 100644
--- a/src/android/camera_request.h
+++ b/src/android/camera_request.h
@@ -34,13 +34,19 @@ public:
 	};
 
 	struct StreamBuffer {
+		StreamBuffer(CameraStream *stream,
+			     const camera3_stream_buffer_t &buffer,
+			     Camera3RequestDescriptor *request);
+		StreamBuffer(StreamBuffer &&other);
+		~StreamBuffer();
+
 		CameraStream *stream;
 		buffer_handle_t *camera3Buffer;
 		std::unique_ptr<libcamera::FrameBuffer> frameBuffer;
 		int fence;
-		Status status;
-		libcamera::FrameBuffer *internalBuffer;
-		const libcamera::FrameBuffer *srcBuffer;
+		Status status = Status::Success;
+		libcamera::FrameBuffer *internalBuffer = nullptr;
+		const libcamera::FrameBuffer *srcBuffer = nullptr;
 		std::unique_ptr<CameraBuffer> dstBuffer;
 		Camera3RequestDescriptor *request;
 	};
-- 
2.31.1