[PATCH v1 2/4] libcamera: controls: Strings are arrays
Barnabás Pőcze
barnabas.pocze at ideasonboard.com
Tue Apr 1 15:19:37 CEST 2025
`ControlId::isArray()` and `ControlValue::isArray()` disagree
in the case of strings. Fix it by setting the static size of a
string to `libcamera::dynamic_extent` to denote a dynamically
sized array-like value.
One unfortunate side effect of this change is that if there were
string controls (there are none at the moment), then `cam` would
display them with an extra `Size: n` annotation.
Bug: https://bugs.libcamera.org/show_bug.cgi?id=255
Signed-off-by: Barnabás Pőcze <barnabas.pocze at ideasonboard.com>
---
include/libcamera/controls.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/libcamera/controls.h b/include/libcamera/controls.h
index c1919d864..d35347f4c 100644
--- a/include/libcamera/controls.h
+++ b/include/libcamera/controls.h
@@ -98,7 +98,7 @@ struct control_type<float> {
template<>
struct control_type<std::string> {
static constexpr ControlType value = ControlTypeString;
- static constexpr std::size_t size = 0;
+ static constexpr std::size_t size = libcamera::dynamic_extent;
};
template<>
--
2.49.0
More information about the libcamera-devel
mailing list