[libcamera-devel] [PATCH 6/8] libcamera: pipeline_handler: Make registerType() private
Jacopo Mondi
jacopo at jmondi.org
Fri Oct 7 16:10:37 CEST 2022
Hi Laurent
On Tue, Oct 04, 2022 at 12:21:26AM +0300, Laurent Pinchart via libcamera-devel wrote:
> The PipelineHandlerFactory::registerType() function is called by the
> PipelineHandlerFactory class only. Make it private.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
here you go!
Reviewed-by: Jacopo Mondi <jacopo at jmondi.org>
Thanks
j
> ---
> include/libcamera/internal/pipeline_handler.h | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/include/libcamera/internal/pipeline_handler.h b/include/libcamera/internal/pipeline_handler.h
> index 46df69dc789a..ebbdf2aa391f 100644
> --- a/include/libcamera/internal/pipeline_handler.h
> +++ b/include/libcamera/internal/pipeline_handler.h
> @@ -108,10 +108,11 @@ public:
>
> const std::string &name() const { return name_; }
>
> - static void registerType(PipelineHandlerFactory *factory);
> static std::vector<PipelineHandlerFactory *> &factories();
>
> private:
> + static void registerType(PipelineHandlerFactory *factory);
> +
> virtual PipelineHandler *createInstance(CameraManager *manager) const = 0;
>
> std::string name_;
> --
> Regards,
>
> Laurent Pinchart
>
More information about the libcamera-devel
mailing list