[PATCH] guides: pipeline-handler: Fix controlInfo_ initialization

Laurent Pinchart laurent.pinchart at ideasonboard.com
Fri Mar 1 11:48:55 CET 2024


On Fri, Mar 01, 2024 at 11:11:35AM +0100, Jacopo Mondi wrote:
> Since commit b48db3c489d3 ("libcamera: controls: Create ControlInfoMap
> with ControlIdMap"), instances of the ControlInfoMap class need to be
> created with an instance of an unordered_map of ControlId to ControlInfo
> and with a ControlIdMap instance.
> 
> The pipeline handler developer guide was never updated to reflect the
> change. Fix it.
> 
> Signed-off-by: Jacopo Mondi <jacopo.mondi at ideasonboard.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

> ---
>  Documentation/guides/pipeline-handler.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/guides/pipeline-handler.rst b/Documentation/guides/pipeline-handler.rst
> index 10b9c75c2a7f..5396eb6d0c56 100644
> --- a/Documentation/guides/pipeline-handler.rst
> +++ b/Documentation/guides/pipeline-handler.rst
> @@ -651,7 +651,7 @@ inline in our VividCameraData init:
>             ctrls.emplace(id, info);
>     }
>  
> -   controlInfo_ = std::move(ctrls);
> +   controlInfo_ = ControlInfoMap(std::move(ctrls), controls::controls);
>  
>  The ``properties_`` field is  a list of ``ControlId`` instances
>  associated with immutable values, which represent static characteristics that can

-- 
Regards,

Laurent Pinchart


More information about the libcamera-devel mailing list