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

Jacopo Mondi jacopo at jmondi.org
Tue Jun 30 12:26:40 CEST 2020


Hi LAurent

On Mon, Jun 29, 2020 at 04:25:21PM +0200, Niklas Söderlund wrote:
> 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>
>

Reviewed-by: Jacopo Mondi <jacopo at jmondi.org>

Thanks
  j

> > ---
> >  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
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel


More information about the libcamera-devel mailing list