<div dir="ltr">Hi Laurent,<div><br></div><div>Thanks for the review, and for making the tweaks to the commit message.  I believe </div><div>that was the only instance, with that patch I was able to build with C++20 for ChromeOS.</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Nov 14, 2023 at 3:52 AM Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com">laurent.pinchart@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Brett,<br>
<br>
Thank you for your first libcamera patch :-)<br>
<br>
On Mon, Nov 13, 2023 at 09:52:44AM -0700, Brett Brotherton via libcamera-devel wrote:<br>
> fix -Wdeprecated-this-capture error when building with c++20 by<br>
> explicity naming this in the capture<br>
<br>
Nitpicking, s/fix/Fix/ and s/capture$/capture./<br>
<br>
No need to resend the patch for this, I'll update the commit message<br>
before pushing.<br>
<br>
Is this the only occurrence of implicit *this capture ? I'm happy we're<br>
not doing that bad :-)<br>
<br>
> Signed-off-by: Brett Brotherton <<a href="mailto:bbrotherton@google.com" target="_blank">bbrotherton@google.com</a>><br>
<br>
Reviewed-by: Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com" target="_blank">laurent.pinchart@ideasonboard.com</a>><br>
<br>
I will run this through build testing and push the patch if no issues<br>
are found.<br>
<br>
> ---<br>
>  src/libcamera/pipeline_handler.cpp | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> <br>
> diff --git a/src/libcamera/pipeline_handler.cpp b/src/libcamera/pipeline_handler.cpp<br>
> index 9c74c6cf..2220cb92 100644<br>
> --- a/src/libcamera/pipeline_handler.cpp<br>
> +++ b/src/libcamera/pipeline_handler.cpp<br>
> @@ -649,7 +649,7 @@ void PipelineHandler::registerCamera(std::shared_ptr<Camera> camera)<br>
>   */<br>
>  void PipelineHandler::hotplugMediaDevice(MediaDevice *media)<br>
>  {<br>
> -     media->disconnected.connect(this, [=]() { mediaDeviceDisconnected(media); });<br>
> +     media->disconnected.connect(this, [=, this]() { mediaDeviceDisconnected(media); });<br>
>  }<br>
>  <br>
>  /**<br>
<br>
-- <br>
Regards,<br>
<br>
Laurent Pinchart<br>
</blockquote></div>