[RFC PATCH v1 07/12] apps: lc-compliance: Use array instead of `std::vector`

[Paul Elder]

On Fri, Dec 20, 2024 at 03:08:33PM +0000, Barnabás Pőcze wrote:
> There is no reason to use `std::vector` for this static
> data, a simple array will do fine.
> 
> Signed-off-by: Barnabás Pőcze <pobrn at protonmail.com>

Reviewed-by: Paul Elder <paul.elder at ideasonboard.com>

> ---
>  src/apps/lc-compliance/tests/capture_test.cpp | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/src/apps/lc-compliance/tests/capture_test.cpp b/src/apps/lc-compliance/tests/capture_test.cpp
> index db196a949..97465a612 100644
> --- a/src/apps/lc-compliance/tests/capture_test.cpp
> +++ b/src/apps/lc-compliance/tests/capture_test.cpp
> @@ -18,8 +18,9 @@ namespace {
>  
>  using namespace libcamera;
>  
> -const std::vector<int> NUMREQUESTS = { 1, 2, 3, 5, 8, 13, 21, 34, 55, 89 };
> -const std::vector<StreamRole> ROLES = {
> +const int NUMREQUESTS[] = { 1, 2, 3, 5, 8, 13, 21, 34, 55, 89 };
> +
> +const StreamRole ROLES[] = {
>  	StreamRole::Raw,
>  	StreamRole::StillCapture,
>  	StreamRole::VideoRecording,
> -- 
> 2.47.1
> 
>