[libcamera-devel] [PATCH 4/4] cam: fix return type of configureStreams()

[Niklas Söderlund]

The only caller of configureStreams() stores its return value as an int
and not bool. This is confusing and also prevents the (possibly)
different error codes returned by Camera::configureStreams() to be
propagated inside the cam tool. Fix this by changing the return type to
int and propagate the return value from the camera.

Signed-off-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
---
 src/cam/main.cpp | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/cam/main.cpp b/src/cam/main.cpp
index 9b67ab75a6a1663e..7c47ce652aa025d1 100644
--- a/src/cam/main.cpp
+++ b/src/cam/main.cpp
@@ -78,7 +78,7 @@ static int parseOptions(int argc, char *argv[])
 	return 0;
 }
 
-static bool configureStreams(Camera *camera, std::vector<Stream *> &streams)
+static int configureStreams(Camera *camera, std::vector<Stream *> &streams)
 {
 	KeyValueParser::Options format = options[OptFormat];
 	Stream *id = streams.front();
@@ -98,10 +98,7 @@ static bool configureStreams(Camera *camera, std::vector<Stream *> &streams)
 			config[id].pixelFormat = format["pixelformat"];
 	}
 
-	if (camera->configureStreams(config))
-		return false;
-
-	return true;
+	return camera->configureStreams(config);
 }
 
 static void requestComplete(Request *request, const std::map<Stream *, Buffer *> &buffers)
-- 
2.20.1