[libcamera-devel] [PATCH 1/2] simple-cam: Turn globals static

[Kieran Bingham]

On Sun, Sep 26, 2021 at 11:39:23PM +0300, Laurent Pinchart wrote:
> Hi Dorota,
> 
> Thank you for the patch.
> 
> On Sun, Sep 26, 2021 at 11:33:18PM +0300, Laurent Pinchart wrote:
> > From: Dorota Czaplejewicz <dorota.czaplejewicz at puri.sm>
> > 
> > This fixes a warning from QtCreator.
> > 
> > Signed-off-by: Dorota Czaplejewicz <dorota.czaplejewicz at puri.sm>
> 
> Looks good to me. I'll submit a corresponding fix to the application
> writer's guide in libcamera.

I've got other patches to submit to that too ... maybe it's time to
revive them.

But this is likely a good thing to do all the same

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

> 
> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> 
> > ---
> >  simple-cam.cpp | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/simple-cam.cpp b/simple-cam.cpp
> > index 95e472b4377a..350f68be7906 100644
> > --- a/simple-cam.cpp
> > +++ b/simple-cam.cpp
> > @@ -16,8 +16,8 @@
> >  #define TIMEOUT_SEC 3
> >  
> >  using namespace libcamera;
> > -std::shared_ptr<Camera> camera;
> > -EventLoop loop;
> > +static std::shared_ptr<Camera> camera;
> > +static EventLoop loop;
> >  
> >  /*
> >   * --------------------------------------------------------------------
> > 
> > base-commit: b8da445fc72cfd436ed7ddb8a428e7ea11af9fd9
> > prerequisite-patch-id: a93394f0289868f6be89478f79cc43f1080d5751
> 
> -- 
> Regards,
> 
> Laurent Pinchart

-- 
--
Kieran