[libcamera-devel] [PATCH v1 2/9] libcamera: ipa_context_wrapper: Fix bad copy&paste in comment

Niklas Söderlund niklas.soderlund at ragnatech.se
Mon Jun 29 16:25:21 CEST 2020


Hi Laurent,

Thanks for your patch.

On 2020-06-29 02:19:27 +0300, Laurent Pinchart wrote:
> A comment copied from the IPAInterfaceWrapper incorrectly mentions the
> IPAInterfaceWrapper class instead of the IPAContextWrapper class. Fix
> it.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Reviewed-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>

> ---
>  src/libcamera/ipa_context_wrapper.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/libcamera/ipa_context_wrapper.cpp b/src/libcamera/ipa_context_wrapper.cpp
> index 054aa6ba2089..471118f57cdc 100644
> --- a/src/libcamera/ipa_context_wrapper.cpp
> +++ b/src/libcamera/ipa_context_wrapper.cpp
> @@ -283,7 +283,7 @@ void IPAContextWrapper::queue_frame_action(void *ctx, unsigned int frame,
>  #ifndef __DOXYGEN__
>  /*
>   * This construct confuses Doxygen and makes it believe that all members of the
> - * operations is a member of IPAInterfaceWrapper. It must thus be hidden.
> + * operations is a member of IPAContextWrapper. It must thus be hidden.
>   */
>  const struct ipa_callback_ops IPAContextWrapper::callbacks_ = {
>  	.queue_frame_action = &IPAContextWrapper::queue_frame_action,
> -- 
> Regards,
> 
> Laurent Pinchart
> 
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel

-- 
Regards,
Niklas Söderlund


More information about the libcamera-devel mailing list