<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Mar 12, 2012, at 5:52 PM, Richard Smith wrote:</div><blockquote type="cite"><div class="gmail_quote">On Mon, Mar 12, 2012 at 5:37 PM, John McCall <span dir="ltr"><<a href="mailto:rjmccall@apple.com">rjmccall@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; ">Modified: cfe/trunk/test/SemaCXX/lambda-expressions.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/lambda-expressions.cpp?rev=152593&r1=152592&r2=152593&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/lambda-expressions.cpp?rev=152593&r1=152592&r2=152593&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/test/SemaCXX/lambda-expressions.cpp (original)<br>
+++ cfe/trunk/test/SemaCXX/lambda-expressions.cpp Mon Mar 12 19:37:01 2012<br>
@@ -139,3 +139,12 @@<br>
unsigned int result = 0;<br>
auto l = [&]() { ++result; };<br>
}<br>
+<br>
+namespace ModifyingCapture {<br>
+ void test() {<br>
+ int n = 0;<br>
+ [=] {<br>
+ n = 1; // expected-error {{variable is not assignable (captured by copy)}}<br></blockquote><div><br></div><div>It would be great if this diagnostic mentioned that the lambda is not marked as 'mutable'.</div>
</div>
</blockquote></div><br><div>I'd been vaguely unhappy with this diagnostic anyway, so I took this prompting to improve it in a couple dimensions. r152598.</div><div><br></div><div>John.</div></body></html>