[RFC PATCH v2 12/16] apps: lc-compliance: Add message to `GTEST_SKIP()`

Barnabás Pőcze pobrn at protonmail.com
Tue Jan 14 19:22:47 CET 2025


Just like other gtest macros, `GTEST_SKIP()` returns an object
to which a formatted message can be added using the usual `<<`
stream operator. So use it instead of printing to `std::cout`.

Signed-off-by: Barnabás Pőcze <pobrn at protonmail.com>
---
 src/apps/lc-compliance/helpers/capture.cpp | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/src/apps/lc-compliance/helpers/capture.cpp b/src/apps/lc-compliance/helpers/capture.cpp
index 91c4d4400..43db15d2d 100644
--- a/src/apps/lc-compliance/helpers/capture.cpp
+++ b/src/apps/lc-compliance/helpers/capture.cpp
@@ -26,10 +26,8 @@ void Capture::configure(StreamRole role)
 {
 	config_ = camera_->generateConfiguration({ role });
 
-	if (!config_) {
-		std::cout << "Role not supported by camera" << std::endl;
-		GTEST_SKIP();
-	}
+	if (!config_)
+		GTEST_SKIP() << "Role not supported by camera";
 
 	if (config_->validate() != CameraConfiguration::Valid) {
 		config_.reset();
@@ -85,10 +83,8 @@ void CaptureBalanced::capture(unsigned int numRequests)
 
 	/* No point in testing less requests then the camera depth. */
 	if (buffers.size() > numRequests) {
-		std::cout << "Camera needs " + std::to_string(buffers.size())
-			+ " requests, can't test only "
-			+ std::to_string(numRequests) << std::endl;
-		GTEST_SKIP();
+		GTEST_SKIP() << "Camera needs " << buffers.size()
+			     << " requests, can't test only " << numRequests;
 	}
 
 	queueCount_ = 0;
-- 
2.48.0




More information about the libcamera-devel mailing list