<div dir="ltr">This was discussed when the warning was added. I felt the same way, but Aaron convinced me that this is short ("short") term pain:<div><a href="http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20170116/183259.html">http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20170116/183259.html</a><br></div><div><a href="https://marc.info/?l=cfe-commits&m=148521057007058&w=2">https://marc.info/?l=cfe-commits&m=148521057007058&w=2</a><br></div><div><a href="https://marc.info/?l=cfe-commits&m=148546938216504&w=2">https://marc.info/?l=cfe-commits&m=148546938216504&w=2</a><br></div><div><br></div><div>We disabled the warning for now in Chromium.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 5, 2018 at 12:50 PM, Tobias Hahn via cfe-dev <span dir="ltr"><<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I did that as well as posting my wish here, let's see...<br>
<span class="HOEnZb"><font color="#888888"><br>
T<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> Am 05.04.2018 um 17:54 schrieb Keane, Erich via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>>:<br>
><br>
> " Since I don't expect MSVC to be standards compliant any time soon,"<br>
><br>
> I think you may wish to report this as a bug to the MSVC team.  They have been REALLY good about fixing standards compliance issues in the past two years or so.<br>
><br>
> -----Original Message-----<br>
> From: cfe-dev [mailto:<a href="mailto:cfe-dev-bounces@lists.llvm.org">cfe-dev-bounces@lists.<wbr>llvm.org</a>] On Behalf Of Tobias Hahn via cfe-dev<br>
> Sent: Thursday, April 5, 2018 8:50 AM<br>
> To: <a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a><br>
> Subject: [cfe-dev] Wish: Split Wunused-lambda-capture<br>
><br>
> In migrating our codebase to Xcode 9.3, I came across the new warning -Wunused-lambda-capture. This seems like a helpful warning to enable; yet unfortunately I cannot due to incompatibility with MSVC.<br>
><br>
> The problem is that it warns on the following code (<a href="https://godbolt.org/g/HDiqR8" rel="noreferrer" target="_blank">https://godbolt.org/g/HDiqR8</a>)<wbr>:<br>
><br>
> int times2(int num) {<br>
>  const int two = 2;<br>
>  auto multiply = [two](int i) { return i * two; };<br>
>  return multiply(num);<br>
> }<br>
><br>
> Per the standard, it is unnecessary to capture `two` because it is an integral constant. MSVC doesn't know about this, and throws C3493 (variable cannot be implicitly captured) if I remove `two` from the capture clause to fix clang's -Wunused-lambda-capture (<a href="https://godbolt.org/g/VNNfqT" rel="noreferrer" target="_blank">https://godbolt.org/g/VNNfqT</a>)<wbr>.<br>
><br>
> Since I don't expect MSVC to be standards compliant any time soon, it would be great if this warning could be split into two warnings:<br>
><br>
> -Wunused-lambda-capture would only warn on truly unused captures, but not captures that are merely unnecessary (but still used as in the example above).<br>
><br>
> -Wunnecessary-lambda-capture would warn on redundant explicit captures (as in the example above).<br>
><br>
> Thanks for considering my wish and making clang even better!<br>
><br>
> Tobias<br>
><br>
> Ableton AG, Schoenhauser Allee 6-7, 10119 Berlin, Germany Sitz (Registered Office) Berlin, Amtsgericht Berlin-Charlottenburg, HRB 72838 Vorstand (Management Board): Gerhard Behles, Jan Bohl Vorsitzender des Aufsichtsrats (Chair of the Supervisory Board): Uwe Struck<br>
><br>
> ______________________________<wbr>_________________<br>
> cfe-dev mailing list<br>
> <a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-dev</a><br>
> ______________________________<wbr>_________________<br>
> cfe-dev mailing list<br>
> <a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-dev</a><br>
<br>
Ableton AG, Schoenhauser Allee 6-7, 10119 Berlin, Germany<br>
Sitz (Registered Office) Berlin, Amtsgericht Berlin-Charlottenburg, HRB 72838<br>
Vorstand (Management Board): Gerhard Behles, Jan Bohl<br>
Vorsitzender des Aufsichtsrats (Chair of the Supervisory Board): Uwe Struck<br>
<br>
<br>
______________________________<wbr>_________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-dev</a><br>
</div></div></blockquote></div><br></div>