[PATCH v2 2/3] libcamera: ipa_proxy: Unify configurationFile argument name

Kieran Bingham kieran.bingham at ideasonboard.com
Wed Jul 31 11:19:46 CEST 2024


Quoting Milan Zamazal (2024-07-08 13:38:02)
> The argument name is different in the declaration and the definition.
> Let's use the same one in both, namely `name'.
> 
> Signed-off-by: Milan Zamazal <mzamazal at redhat.com>


Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

> ---
>  include/libcamera/internal/ipa_proxy.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/libcamera/internal/ipa_proxy.h b/include/libcamera/internal/ipa_proxy.h
> index ed6a5bcf..5240f69f 100644
> --- a/include/libcamera/internal/ipa_proxy.h
> +++ b/include/libcamera/internal/ipa_proxy.h
> @@ -31,7 +31,7 @@ public:
>  
>         bool isValid() const { return valid_; }
>  
> -       std::string configurationFile(const std::string &file) const;
> +       std::string configurationFile(const std::string &name) const;
>  
>  protected:
>         std::string resolvePath(const std::string &file) const;
> -- 
> 2.44.1
>


More information about the libcamera-devel mailing list